Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Compatibility with latest Ampache 5 db updates #47

Closed
sandersantema opened this issue May 28, 2021 · 9 comments
Closed

Compatibility with latest Ampache 5 db updates #47

sandersantema opened this issue May 28, 2021 · 9 comments

Comments

@sandersantema
Copy link
Contributor

I was wondering whether amperfy is compatible with the latest Ampache 5 db updates which were merged 3 days ago:
ampache/ampache#2926

I could imagine that this would only be a backend change which would change any API details but I don't know for sure and I'd like to know before upgrading my Ampache server to the latest version.

@BLeeEZ
Copy link
Owner

BLeeEZ commented May 28, 2021

I will check this next week.

@lachlan-00
Copy link

If you have any trouble please tag me directly and I'll I'll help. The server changes are big but they are not actually breaking to an external client.

@BLeeEZ
Copy link
Owner

BLeeEZ commented May 31, 2021

I am running Ampache via docker.
In the currently uploaded docker image the mentioned commit is not included.
To test the commit I changed my docker-compose file from

  ampache-app:
    image: ampache/ampache:develop

to

  ampache-app:
    build: https://github.com/ampache/ampache-docker.git#develop

With that I tried to build the current image by myself.
After executing docker-compose build I get the following error message:

Loading composer repositories with package information
Installing dependencies (including require-dev) from lock file
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - Installation request for scn/phptal v3.1.0 -> satisfiable by scn/phptal[v3.1.0].
    - scn/phptal v3.1.0 requires ext-gettext * -> the requested PHP extension gettext is missing from your system.

  To enable extensions, verify that they are enabled in your .ini files:
    - /usr/local/etc/php/php-cli.ini
    - /usr/local/etc/php/conf.d/docker-php-ext-intl.ini
    - /usr/local/etc/php/conf.d/docker-php-ext-sodium.ini
    - /usr/local/etc/php/conf.d/docker-php-ext-zip.ini
  You can also run `php --ini` inside terminal to see which files are used by PHP in CLI mode.
ERROR: Service 'ampache-app' failed to build: The command '/bin/sh -c tar -xzf /tmp/develop.tar.gz --strip=1 -C .     &&  apk add --no-cache php-intl icu-dev     &&  docker-php-ext-install intl     &&  composer install --prefer-source --no-interaction     &&  rm -rf .git* .php_cs .sc .scrutinizer.yml .tgitconfig .travis.yml .tx *.md     &&  mv ./public/rest/.htac* ./public/rest/.htaccess     &&  mv ./public/play/.htac* ./public/play/.htaccess     &&  mv ./public/channel/.htac* ./public/channel/.htaccess     &&  chmod -R 775 .' returned a non-zero code: 2

@lachlan-00
Copy link

i'll be rebuilding this week, just prepping a new point release for the master branch and will redo develop

@lachlan-00
Copy link

in about 60000 ish seconds my poor mysql box will have built a new develop docker
image

@lachlan-00

This comment has been minimized.

@BLeeEZ
Copy link
Owner

BLeeEZ commented Jun 1, 2021

@lachlan-00 Thank you for your support
@sandersantema I updated the Ampache server. As far as I can see Amperfy runs fine with the updated version.

@BLeeEZ
Copy link
Owner

BLeeEZ commented Jun 2, 2021

I encountered a issue with "artist_albums" and reported it directly on Ampache: ampache/ampache#2932

Due to this bug all albums will display "0 songs" after a fetch.

@sandersantema
Copy link
Contributor Author

I've updated as well, no problems so far 🎉

@BLeeEZ BLeeEZ closed this as completed Jun 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants