Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 211985 - www-apps/mod_survey-3.2.5 version bump
Summary: www-apps/mod_survey-3.2.5 version bump
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High enhancement
Assignee: Gentoo Web Application Packages Maintainers
URL: http://www.modsurvey.org
Whiteboard:
Keywords:
Depends on:
Blocks: 150255
  Show dependency tree
 
Reported: 2008-03-01 15:32 UTC by Joel Palmius
Modified: 2008-03-04 12:03 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments
new ebuild (mod_survey-3.2.5.ebuild,1.65 KB, text/plain)
2008-03-01 15:35 UTC, Joel Palmius
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Joel Palmius 2008-03-01 15:32:30 UTC
There is a new stable release of Mod_Survey available. I'm attaching a new ebuild. 

Reproducible: Always




Because of prior problems with sed:ing and piping info through the installer script, I've repackaged this mod_survey release in a tarball specifically aimed at gentoo. This reduces the need for sed:ing and other interventions to one (setting default language based on LINGUAS). 

I'm a noob at ebuild writing so please bear with me here. I'm strongly suspecting that I'm breaking some policies by not relying on web-apps code, but I did not understand what I was doing, so I made the ebuild as minimal as I could. Thus the ebuild will most likely need to be prodded by someone with more portage insight than me. 

This said, it works for me.
Comment 1 Joel Palmius 2008-03-01 15:33:39 UTC
This supercedes 150255, as I don't think bug is valid anymore. 
Comment 2 Joel Palmius 2008-03-01 15:35:29 UTC
Created attachment 145015 [details]
new ebuild

Minimalist ebuild relying on re-packaged version of mod_survey
Comment 3 Benedikt Böhm (RETIRED) gentoo-dev 2008-03-04 09:40:55 UTC
(In reply to comment #2)
> Created an attachment (id=145015) [edit]
> new ebuild
> 
> Minimalist ebuild relying on re-packaged version of mod_survey

i don't think using this tarball is a good idea, as we cannot/don't want to rely on a special gentoo tarball provided by upstream, especially since the tarball layout is wrong ... besides you removed the webapp functionality from the attached ebuild

3.2.5 with some path updates and webroot cleanup in cvs now
Comment 4 Benedikt Böhm (RETIRED) gentoo-dev 2008-03-04 09:46:45 UTC
(In reply to comment #0)
> This reduces the need for sed:ing and other interventions to one
> (setting default language based on LINGUAS). 

i somehow missed that you still use sw instead of sv, but uncommented the sv handling LINGUAS ... maybe it would be a good idea to fix this in the release tarball
Comment 5 Joel Palmius 2008-03-04 10:19:04 UTC
Very well. But please add info on where the file system layout is wrong.

Also, I re-inserted sv in the ebuild since it was fixed in the re-pack (simply added symlink sv.sl -> sw.sl). All languages are installed by the installer script anyway: the option in the installer is for default language, not what language to install. 

Since the survey.conf file will always be the same for gentoo systems, it might be a better idea to use the "mkgentootarball.pl" script instead of the "installer.pl" script, probably as a part of the build section of the ebuild. Once the mk script is fixed with correct path locations, that would make the ebuild rather less complex. 
Comment 6 Benedikt Böhm (RETIRED) gentoo-dev 2008-03-04 10:34:28 UTC
(In reply to comment #5)
> Very well. But please add info on where the file system layout is wrong.

- examples are documentation
- documentation belongs to /usr/share/doc/${PF}
- you should not install directly into /var/www

> Also, I re-inserted sv in the ebuild since it was fixed in the re-pack (simply
> added symlink sv.sl -> sw.sl).

but as sw is wrong, you should fix it in the "normal" tarball :)

> Since the survey.conf file will always be the same for gentoo systems, it might
> be a better idea to use the "mkgentootarball.pl" script instead of the
> "installer.pl" script, probably as a part of the build section of the ebuild.
> Once the mk script is fixed with correct path locations, that would make the
> ebuild rather less complex. 

maybe you can add some command-line options for the installer, this would get rid of the sed-fu and maybe also /dev/null workaround?
Comment 7 Joel Palmius 2008-03-04 12:03:28 UTC
> but as sw is wrong, you should fix it in the "normal" tarball :)

Yup, this will be fixed in next release. However, no-one reported this bug upstreams, so I only saw it when I sat down and tried to update the ebuild for the already released 3.2.5. And then only as a comment inside an ebuild I had never seen before.