Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 84733 - net-ftp/vsftpd ebuild cleanup and pam fixes
Summary: net-ftp/vsftpd ebuild cleanup and pam fixes
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All All
: High normal (vote)
Assignee: Roy Marples (RETIRED)
URL:
Whiteboard:
Keywords:
: 90591 95018 (view as bug list)
Depends on:
Blocks: 79024 93119
  Show dependency tree
 
Reported: 2005-03-10 03:51 UTC by Diego Elio Pettenò (RETIRED)
Modified: 2005-06-07 11:42 UTC (History)
5 users (show)

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


Attachments
Patch over current ebuild (vsftpd-2.0.1.patch,2.69 KB, patch)
2005-03-10 03:51 UTC, Diego Elio Pettenò (RETIRED)
Details | Diff
Ebuild patch (vsftp-2.0.2.patch,3.11 KB, patch)
2005-05-12 06:01 UTC, Diego Elio Pettenò (RETIRED)
Details | Diff
pamd file (vsftpd.pam-include,335 bytes, text/plain)
2005-05-12 06:02 UTC, Diego Elio Pettenò (RETIRED)
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Diego Elio Pettenò (RETIRED) gentoo-dev 2005-03-10 03:51:08 UTC
The attached patch applied over current ebuild clean up vsftpd ebuild:

- Changes the dependency on openssl to 0.9.7d instead of 0.9.7d-r1 (needed for fbsd profile to work, but still safe for linux, as -r1 fixes only an smime problem which surely doesn't affect vsftpd as no smime string can be found on sources nor scripts).
- Removes ucspi-tcp dependency, as the site, nor the sources, nor the scripts refers at it. Maybe is a legacy dependency of the 1.x series, but surely isn't needed at the moment.
- Adds an xinetd use-flag which adds optional runtime dependency on xinetd, and is used to install the xinetd.d script only if requested.
- Removes a commented-out filter-flag which is no more needed (at least so states changelog)
- Change an if ! use; then; fi block into a use || as it's a single line (as for the two use && above
- Removes the einfo about disable pam support on compile (it's quite rare that an user can read what is stated before compile, and anyway as pam is enabled by default the user should know that pam support is disabled looking at the flags)
- Cleans up the installation of files, removing a 'cp -a' instance (it won't work on non-gnu systems) and replacing it with a cp -R (why should we use preserve permission and then change the permissions to make them root-owned?)
- Installs pam files only if pam support is requested
- Removes the install call for empty dirs in postinst, and replace it with keepdirs
- Removes depscan call (should be done automatically).

Please note that version 2.0.2 is out, it builds fine using the normal ebuild (patched with the attached patch).

HTH, Diego
Comment 1 Diego Elio Pettenò (RETIRED) gentoo-dev 2005-03-10 03:51:26 UTC
Created attachment 53067 [details, diff]
Patch over current ebuild
Comment 2 Jakub Moc (RETIRED) gentoo-dev 2005-04-27 09:01:42 UTC
*** Bug 90591 has been marked as a duplicate of this bug. ***
Comment 3 Diego Elio Pettenò (RETIRED) gentoo-dev 2005-05-12 06:01:53 UTC
Created attachment 58744 [details, diff]
Ebuild patch

The attached, updated patch fixes pamd file to be more compatible with g/fbsd
which uses the include directive instead of pam_stack.so.
It also changes the dependency on pam 0.78 at least.

(see also the next attachment which is the pamd file itself)
Comment 4 Diego Elio Pettenò (RETIRED) gentoo-dev 2005-05-12 06:02:46 UTC
Created attachment 58745 [details]
pamd file
Comment 5 Diego Elio Pettenò (RETIRED) gentoo-dev 2005-05-12 06:04:02 UTC
CCed bsd as the pamd file is needed by g/fbsd.
Comment 6 Dominik Stadler (RETIRED) gentoo-dev 2005-05-19 15:11:55 UTC
latest available version is 2.0.3 now.
Comment 7 Jakub Moc (RETIRED) gentoo-dev 2005-06-04 06:16:10 UTC
*** Bug 95018 has been marked as a duplicate of this bug. ***
Comment 8 Roy Marples (RETIRED) gentoo-dev 2005-06-07 11:42:47 UTC
Fixed in vsftpd-2.0.3