Bug 205504 - [REVIVE PORT] audio/ampache version 3.8.1
Summary: [REVIVE PORT] audio/ampache version 3.8.1
Status: Closed Feedback Timeout
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: freebsd-ports-bugs (Nobody)
URL: https://github.com/johnko/freebsd-por...
Keywords: feature, needs-patch, needs-qa
Depends on:
Blocks:
 
Reported: 2015-12-22 06:16 UTC by git
Modified: 2018-08-09 11:44 UTC (History)
2 users (show)

See Also:


Attachments
poudriere log (25.73 KB, text/plain)
2016-07-19 08:44 UTC, Andrej Ebert
no flags Details
poudriere-portlint log (1.03 KB, text/plain)
2016-07-19 08:47 UTC, Andrej Ebert
no flags Details
diff against svn (474.20 KB, patch)
2016-07-19 09:09 UTC, Andrej Ebert
no flags Details | Diff
poudriere log (24.04 KB, text/plain)
2016-07-19 09:09 UTC, Andrej Ebert
no flags Details
poudriere-portlint log (597 bytes, text/plain)
2016-07-19 09:10 UTC, Andrej Ebert
no flags Details
svn diff for version 3.8.2 - without composer (486.38 KB, patch)
2016-07-19 12:05 UTC, Andrej Ebert
no flags Details | Diff
poudriere log without composer (19.86 KB, text/plain)
2016-07-19 12:06 UTC, Andrej Ebert
no flags Details
poudriere-portlint log without composer (188 bytes, text/plain)
2016-07-19 12:07 UTC, Andrej Ebert
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description git 2015-12-22 06:16:06 UTC
Is it proper to call `composer` in the Makefile?

It appears to be needed to install other dependencies.

Link to Makefile line with `composer`:

    https://github.com/johnko/freebsd-ports-ampache/blob/af964982682a310fe3284ff3c4e986cd947ac7ec/audio/ampache/Makefile#L47


Link to Makefile, distinfo, pkg-plist, etc.:

    https://github.com/johnko/freebsd-ports-ampache


With that Makefile and etc., this works:

    make && make install && make package
Comment 1 VK freebsd_triage 2015-12-22 09:44:10 UTC
Thanks for the submission.

Can you please attach a patch against the ports' svn tree? The process of revival is described here (steps 1-6):

https://www.freebsd.org/doc/en_US.ISO8859-1/articles/committers-guide/ports.html#ports-qa-re-adding

The port was removed in revision 402562, so you will need the one before that:

https://svnweb.freebsd.org/ports?view=revision&revision=402562

Also please test the port with portlint and poudriere.

* CC-ing previous maintainer in case they're interested.
Comment 2 git 2015-12-25 10:27:38 UTC
I found an issue in my pkg-plist where I blindly added all files that php-composer fetched. I fixed part of it by manually excluding anything matching .*/.git/.*

I'm worried that php-composer could pull different versions of files than those listed in pkg-plist because it pulls from master on some dependency's repos.

I'm running some more tests before I submit a final patch.
Comment 3 Andrej Ebert 2016-07-19 08:44:29 UTC
Created attachment 172700 [details]
poudriere log

Here's a poudriere test log for the port that's on github at the moment.
There are some warnings for the Makefile:
WANT_PHP_WEB is deprecated, please use USES=php:web
Using USE_PHP alone is deprecated, please use USES=php

And composer fails to fetch anything, I think beacause it is not allowed in the stage phase?
Comment 4 Andrej Ebert 2016-07-19 08:47:14 UTC
Created attachment 172701 [details]
poudriere-portlint log

Also some portlint warnings and errors.
Comment 5 Andrej Ebert 2016-07-19 09:09:37 UTC
Created attachment 172704 [details]
diff against svn

Here's a svn diff with some of the portlint errors and make warnings fixed. composer still can't fetch anything, though.
Comment 6 Andrej Ebert 2016-07-19 09:09:57 UTC
Created attachment 172705 [details]
poudriere log
Comment 7 Andrej Ebert 2016-07-19 09:10:18 UTC
Created attachment 172706 [details]
poudriere-portlint log
Comment 8 Andrej Ebert 2016-07-19 09:11:36 UTC
What is the proper way of using php-composer in ports anyway? Couldn't find any docs/examples.
Comment 9 Andrej Ebert 2016-07-19 12:05:39 UTC
Created attachment 172713 [details]
svn diff  for version 3.8.2 - without composer

So ampache also has a release with all the dependencies already in it, without the need for composer:

ampache-[version]_all.zip, see https://github.com/ampache/ampache/wiki/Installation#composer

I made modified the port for this version (without depending on composer and git), fixed some things, it builds, installs and deinstalls fine, but haven't tested running it yet.

Also, I think mysql-server has to be added as a dependency, since ampache need a MySQL instance.

The pkg-plist ist (mostly) auto-generated, and I'm not so sure if the permissions are right on all the files/folders.

All in all it needs further work and testing, but should be a good starting point without dealing with composer.
Comment 10 Andrej Ebert 2016-07-19 12:06:25 UTC
Created attachment 172714 [details]
poudriere log without composer
Comment 11 Andrej Ebert 2016-07-19 12:07:22 UTC
Created attachment 172715 [details]
poudriere-portlint log without composer
Comment 12 Tobias Kortkamp freebsd_committer freebsd_triage 2018-05-05 01:46:46 UTC
Is this still relevant?  If yes then this needs an update to 3.8.6 and
somebody should put this up for review on https://reviews.freebsd.org/.
Comment 13 Andrej Ebert 2018-05-05 09:20:45 UTC
I'll try to make an updated port/diff this weekend. How do I submit something for rewiew?
Comment 14 Tobias Kortkamp freebsd_committer freebsd_triage 2018-05-06 16:26:49 UTC
(In reply to Andrej Ebert from comment #13)
See https://wiki.freebsd.org/Phabricator
Comment 15 Tobias Kortkamp freebsd_committer freebsd_triage 2018-05-20 03:33:04 UTC
Ping