Bugzilla Bug 101505

Please add this excellent webapp to Portage. Thanks!

v0.11.20 (2005 Aug 21) out. Adds https support, and bugfixes.

Created an attachment (id=66953) [details]
zina-0.11.20.ebuild (I had it under www-apps/zina)

Created an attachment (id=66954) [details]
files/zina-0.11.20-webinstall.txt

OK, I added an extremely simple ebuild for zina 0.11.20, and some post-install
instructions (more like advice...). Hope this gets into Portage - I really love
Zina - so simple, but so great.

(Oh, and I tested it with some Amon Tobin ;). Works great!)

Created an attachment (id=66956) [details]
files/zina-0.11.20-webinstall.txt (with security notice)

I added a security notice. Zina doesn't have any internal user/pass for access
- only an "admin" user. So, beware, and restrict access.

Thanks for the ebuild. A few notes:

1. You should not hardcode version into SRC_URI - use ${PV} or ${P}
2. It should RDEPEND on virtual/php instead of dev-php/php. BTW, did you really
mean php and not mod_php? (If it needs mod_php, then make it virtual/httpd-php
instead.)
3. Empty DEPEND is redundant.

Please, fix the above errors and reopen the bug then.

Oh, and keywords do not follow the policy, correct that as well.

Created an attachment (id=66963) [details]
zina-0.11.20.ebuild (corrected)

Ack, apologies. I made a quick copy and edit of the extant
"www-apps/ampache-3.3.1.4" ebuild - I should have been more careful. Thanks,
Jakub, for taking the time to parse these files and redline the errors. I'm
attaching my fixed ebuild. Also, I'm not sure where to point the SRC_URI when
using SourceForge mirrors, so I pointed it to EasyNews. Please correct me if
that is wrong.

Reopening for further inspection. Thanks.

Oh, and I changed the RDEPEND as requested, using "virtual/httpd-php". However,
the upstream requirements are specfically for (mod_)php-4.1.X. Does this virtual
package handle that?

(In reply to comment #9)

> Ack, apologies. I made a quick copy and edit of the extant
> "www-apps/ampache-3.3.1.4" ebuild - I should have been more careful.

No problem. ;)

> Also, I'm not sure where to point the SRC_URI when
> using SourceForge mirrors, so I pointed it to EasyNews. Please correct me if
> that is wrong.

Probably SRC_URI="mirror://sourceforge/${PN}/${P}.tar.gz", see if that works.

Created an attachment (id=66965) [details]
zina-0.11.20.ebuild (2nd correction - add sourceforge mirror SRC_URI

Thanks, Jakub. Here's the corrected ebuild. Let me know if anything else smells
fishy. :)

Looks good now, thanks. Leaving this for webapps guys. If you could test the
ebuild w/ php-5 and let us know if it works or not, it would be really
appreciated. I'd recommend our Gentoo PHP overlay [1] for testing, that's what
will be coming to portage soon.

And thanks for discovering the broken ampache dependency, on our TODO list now.

[1] http://svn.gnqs.org/projects/gentoo-php-overlay/

I'll give this with PHP 5 a try later today.

Well, sorry, I couldn't get PHP5 to work on my dev box, so it'll have to wait
for another day to test PHP5. So, we only have it confirmed to work on PHP 4.X.

Any chance of this getting added soon? Or is more reviewing needed? Thanks.

Nice ebuild. Had no problem to use the app with php 5.0.5.

Re #18: Hey, thanks for testing! :)

v0.11.21 is out. I just bumped the version rev in this ebuild, and it works fine
for me. Can't upload a new ebuild as I'm on the road and my laptop's /var
partition just bit the dust (i.e. Portage is toast). :( If required, I'll upload
it from my workstation when I get home on the 27th, but, again, version bumping
the ebuild name did the trick.

Any hope of this getting into Portage?

P.S. I'll maintain it, if needed.

Created an attachment (id=72151) [details]
zina-0.11.22.ebuild

Bumped, tested, upgraded fine (version shows on web page; streams). Only
SRC_URI change.

Just cp files/zina-0.11.20-webinstall.txt files/zina-0.11.22-webinstall.txt -
no
changes there.

zina is now in our unofficial overlay:
http://svn.gnqs.org/projects/gentoo-webapps-overlay/browser

Please note that the overlay is unofficial, not a Gentoo project, and not
supported. It is intended to provide easier access to new web applications.

Accidental closure?

(In reply to comment #25)
> Accidental closure?

No, would you please read comment #24?

Hmm... I'd like to continue to track this ebuild until it is included in
Portage with Bugzilla's automated e-mails. How do I do this?

we'll post back to this bug if/when zina is in Portage

Hi,

thanks for the ebuild. Had no problems installing the app. 

The ebuild did try to set the config file as server-owned and config-owned.
This is currently not (yet) supported by webapp-config. I considered it more
important to have the file config-protected, so I removed the "server-owned"
flag. I also added a cache directory which I needed for saving playlists. See
http://svn.gnqs.org/projects/gentoo-webapps-overlay/changeset/76 

Can you confirm that these changes are ok? Since I started using the app, I
will probably maintain it within the portage tree.

in portage

Hmm... not that I have sour grapes, but could I get a little credit for this
ebuild in the ChangeLog? I wrote to the original person who took this out of
the main Portage bugzilla, who is no longer the maintainer on this package, but
had no response. Thanks.

> Hmm... not that I have sour grapes, but could I get a little credit for this
> ebuild in the ChangeLog? I wrote to the original person who took this out of
> the main Portage bugzilla, who is no longer the maintainer on this package, but
> had no response. Thanks.

Mmm - would that be me? Spam filter must be malfunctioning. 
wrobel - would you please add jesse to the changelog next time you touch it?

No it was me Renat.

You just transferred zina to our overlay. At that time the ebuild itself still
had the original author in the ebuild code. I had to remove the original
comments we use in our overlay when I transferred the thing to portage and
forgot to add it to the ChangeLog. 

Sorry about that. Will be fixed on the next commit.