~mediagoblin

Trackers

~mediagoblin/mediagoblin

Last active a month ago

#12 Configure step fails on Arch Linux with syntax error near unexpected token `newline' 1 year, 8 months ago

Comment by ~mediagoblin on ~mediagoblin/mediagoblin

Great, thanks Charlotte! Could you please tell me what version of autoconf you have by running autoconf --version?

#15 Copy contributing docs over from wiki to sphinx 1 year, 8 months ago

Ticket created by ~mediagoblin on ~mediagoblin/mediagoblin

We need high-quality contributor docs and we don't have any.

#14 Does pyconfigure even make sense anymore? 1 year, 8 months ago

Ticket created by ~mediagoblin on ~mediagoblin/mediagoblin

The bootstrap.sh/configure/make steps are pretty unusual for a Python application. These are based on pyconfigure by Brandon Invergo.

I suspect we may be able to add a lot of clarity and remove a significant barrier to contribution by looking at an alternative, but I'm not fully across the background here.

Pyconfigure currently:

  • updates git submodules (bootstrap.sh)
  • looks for a python interpreter and virtualenv package
  • makes a virtualenv
  • installs python dependencies into the virtualenv (./bin/python setup.py develop --upgrade) - why does it reinstall things you already have a system site packages?
  • runs devtools/update_extlib.sh to install Bower and then bower install jquery etc.
  • builds translations
  • builds texinfo docs (broken and we don't use anyway)

That's all pretty unusual for a Python application, and adds a lot of fairly dense build config/infrastructure.

The translation building makes sense, and it's nice that the system can find the python interpreter, but beyond that these tools aren't normally used to create virtualenvs and install Python/JS deps. That's a bit unusual.

Noting that the last release of pyconfigure was 0.2.3 in 2016. The pyconfigure website says that the project has been decommissioned.

Installing could be a compelling feature, if we're not installing as a Python package. But maybe a python package would be easier.

I wonder if it's worth chatting to Brandon Invergo about why it was decommissioned?

#13 View all users media link is hard to find 1 year, 8 months ago

Ticket created by ~mediagoblin on ~mediagoblin/mediagoblin

The only way to get to the "view all a user's media" page is via a hard to spot link at the bottom of the profile page below the first page of media.

This link should be to below the profile text and collections link (if applicable).

It's not so clear what the difference between the profile and the "all media" pages are since they both show media. Maybe the profile should only show the most recent items?

#12 Configure step fails on Arch Linux with syntax error near unexpected token `newline' 1 year, 8 months ago

Ticket created by ~mediagoblin on ~mediagoblin/mediagoblin

lilmike reported by IRC that:

So I've got a really strange issue. I'm trying to run mediagoblin on arch
linux, and running ./bootstrap.sh works fine, but upon running configure I
get
checking for python3... python3
./configure: line 1934: syntax error near unexpected token `newline'
./configure: line 1934: ` '''
This has been happening even before the new release, it's just so strange

Ben suspects this is related to the autoconf version as has seen the same in Guix. Specifying autoconf2.69 in Guix fixes the issue.

#5 --system-site-packages is hard-coded, so why do we specify it during deployment? 1 year, 8 months ago

Comment by ~mediagoblin on ~mediagoblin/mediagoblin

Fixed in 0db9b5bb by removing the VIRTUALENV_FLAGS='--system-site-packages' from docs and CI builds.

REPORTED RESOLVED FIXED

#11 [PATCH] Testing. 1 year, 8 months ago

on ~mediagoblin/mediagoblin

REPORTED RESOLVED INVALID

#5 --system-site-packages is hard-coded, so why do we specify it during deployment? 1 year, 8 months ago

Comment by ~mediagoblin on ~mediagoblin/mediagoblin

I've just run:

make distclean
./bootstrap
./configure
make

and confirmed that make uses --system-site-packages. I think we're safe to remove it from our docs and CI builds.

#4 Replace werkzeug.contrib.atom.AtomFeed with something else 1 year, 9 months ago

Comment by ~mediagoblin on ~mediagoblin/mediagoblin

Done! werkzeug.contrib.atom.AtomFeed has been replaced with FeedGenerator and we can now move beyond Werkzeug 1.0.

REPORTED RESOLVED FIXED

#6 Switch from Celery to RQ 1 year, 9 months ago

Comment by ~mediagoblin on ~mediagoblin/mediagoblin

After fixing the issue of media being marked as failed on celery restart and also the missing celery logging, things aren't looking too bad. Maybe this is not a problem unless we hit another major celery issue.