~peter_horvath

Munich, Germany


#33 Login error: TypeError in basic auth's use of bcrypt a month ago

Comment by ~peter_horvath on ~mediagoblin/mediagoblin

On the channels I tried to contact them, the mailing list was the the most responsive until now. Afaik the problem is that only 1 dev is available and even he is busy. I have also seen a that Olivier Mehani has many unmerged branches in his forked repo ( https://git.sr.ht/~shtrom/mediagoblin ).

#17 Debian installer script a month ago

Comment by ~peter_horvath on ~mediagoblin/mediagoblin

I think it would not be hard to create a debian packaging. Getting an injection into the official debian (ubuntu) repos might be much more hard.

#14 Does pyconfigure even make sense anymore? a month ago

Comment by ~peter_horvath on ~mediagoblin/mediagoblin

I believe the configure script is useful to attract developers accustomed to the gnu, but not to the python world. And the current script is not a big one, effectively it is just a wrapper around the python setuptools. It could be a wrapper also around PyConfigure with minimal work.

#33 Login error: TypeError in basic auth's use of bcrypt a month ago

Comment by ~peter_horvath on ~mediagoblin/mediagoblin

Before I have found this ticket, also I have found (and fixed) the problem. I second, the patch looks correct. [I intentionally do not link my equivalent patch because I want a quick merge].

#35 Suggested minor gitignore updates a month ago

Comment by ~peter_horvath on ~mediagoblin/mediagoblin

#35 Suggested minor gitignore updates a month ago

Ticket created by ~peter_horvath on ~mediagoblin/mediagoblin

After a full compilation, there are intermediary files created, which are not in the .gitignore. Namely:

  1. package-lock.json and package.json (generated by npm)
  2. pyvenv.cfg (generated by virtualenv)

Beside them, install-sh is not in the gitignore, while it is automatically created by the autoconf.

This patch improves these in the .gitignore: https://gitlab.com/peter.horvath/mediagoblin/-/commits/gitignore-fix/ Please merge.

#12 Configure step fails on Arch Linux with syntax error near unexpected token `newline' a month ago

Comment by ~peter_horvath on ~mediagoblin/mediagoblin

Hi, I tracked down the problem to the embedded IF clause in configure.ac . I suggest to apply my change from here: https://gitlab.com/peter.horvath/mediagoblin/-/commits/bugfix/12

The problem is that the embedded AS_IF macro seems not working with newer gnu autoconf. I did not track, exactly why, possibly it is intentional. Most importantly, we just don't need it, because AS_IF already supports the syntax what we want to.