Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 5408 - proposed patch for mailman-2.0.12.ebuild
Summary: proposed patch for mailman-2.0.12.ebuild
Status: RESOLVED WONTFIX
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: x86 Linux
: High normal (vote)
Assignee: Ben Lutgens (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2002-07-22 14:48 UTC by Pau Oliva
Modified: 2003-04-18 18:00 UTC (History)
2 users (show)

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


Attachments
mailman-2.0.12.ebuild.diff (mailman-2.0.12.ebuild.diff,3.40 KB, patch)
2002-07-22 14:48 UTC, Pau Oliva
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Pau Oliva 2002-07-22 14:48:02 UTC
Attached is a patch for mailman-2.0.12.ebuild (still masked at the moment 
of writing this).

This is what the patch basically does:

 * add dependency on virtual/cron
 * verify that filesystem is not mounted with "nosuid" option
   (thus mailman will not work!)
 * install missing mailman images into apache icons directory
   (as sugested by mailman's documentation)
 * in pkg_config:
     - add the ability of automagically set up the cron jobs
       for user mailman in pkg_config
     - test if user is using smrsh in sendmail and link
       Mailman's wrapper
Comment 1 Pau Oliva 2002-07-22 14:48:51 UTC
Created attachment 2468 [details, diff]
mailman-2.0.12.ebuild.diff
Comment 2 Patrick Naubert 2002-09-03 21:49:52 UTC
The "qfiles" dir is not created.
Comment 3 scott.claridge 2002-12-09 22:11:45 UTC
How about rewriting the apache config file based on the INSTALLDIR variable?
A la:
     sed -e "s:/var/mailman:${INSTALLDIR}:" \
        ${FILESDIR}/mailman.conf > mailman.conf

(Also, how about allowing INSTALLDIR to be set from the command line, rather
than direct ebuild edit? It may be against the ebuild philosophy, but it makes a
lot easier than editing ebuilds - even with local overlays.
Maybe like:
  if [ -z ${INSTALLDIR} ]; then
      INSTALLDIR="/var/mailman"
  fi
)
Comment 4 Jeff Terrell 2002-12-09 22:54:25 UTC
Patrick, could you be more precise please?  I could find no reference to
"qfiles" in the mailman documentation, portage programmers how-to, or in the
ebuild itself.  If there still is an issue with Pau's patchfile, I'd like to
contribute to fixing it somehow, but I need more information on what's wrong
with it.

I patched the ebuild with Pau's patchfile, and it seemed to install like a
charm.  Is there a reason we can't unmask this?  I would certainly like to see
mailman in the portage tree, as I have need of it.

This might be unrelated, but how can I remove mailman from the masked packages
list?  I checked and double-checked the /usr/portage/profiles/package.mask file,
and mailman never showed up.
Comment 5 Nick Hadaway 2003-01-21 19:15:02 UTC
NOTE: mailman-2.1 has been added to portage (currently marked unstable)
Please test this new version.  Once all problems have been resolved it should be marked stable.
Comment 6 Martin Holzer (RETIRED) gentoo-dev 2003-04-18 18:00:03 UTC
GLSA 2.1.1 should be used