Bug 127809 - [patch] security/cyrus-sasl2-saslauthd: install man8/saslauthd.8 instead of cat8/saslauthd.8
Summary: [patch] security/cyrus-sasl2-saslauthd: install man8/saslauthd.8 instead of c...
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: Normal Affects Only Me
Assignee: Hajimu UMEMOTO
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-10-03 07:40 UTC by Scot Hetzel
Modified: 2008-10-13 16:20 UTC (History)
0 users

See Also:

Attachments
file.diff (3.49 KB, patch)
2008-10-03 07:40 UTC, Scot Hetzel 		no flags Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Scot Hetzel 2008-10-03 07:40:00 UTC 
While making a backup package of cyrus-sasl2-saslauthd, pkg-create complained about a missing /usr/local/man/cat8/saslauthd.8.gz.

I found that /etc/periodic/weekly/330.catman (with weekly_catman_enable=YES0 will cause the PREFIX/man/cat8/saslauthd.8.gz file to be removed., when it fails to find a corresponding PREFIX/man/man8/saslauthd.8{,.gz} file.

I noticed that ports do-build target removes ${WRKSRC}/saslauthd/saslauthd.8, which causes the saslauthd.8 to be rebuilt using nroff -mdoc saslauthd.mdoc, when ${WRKSRC}/saslauthd/Makefile is used to install this files.

A much cleaner solution would be to install saslauthd.mdoc to PREFIX/man/man8/saslauthd.8, and use the MAN8 variable in the ports Makefile.

After I had modified the port to do this, I compared the man generated cat8/saslauthd.8 file to the nroff -mdoc generated saslauthd.8 file.  The only difference I found involved white space changes between these files.

Fix: Install saslauthd.mdoc in PREFIX/man/man8/saslauthd.8

The attached patch provides this change, as well as some minor fixes.

NOTE: The patch affects both the security/cyrus-sasl2 and security/cyrus-sasl2-saslauthd ports.

Patch attached with submission follows:
How-To-Repeat: Install the port, set weekly_catman_enable=YES in /etc/periodic.conf, run /etc/periodic/weekly/330.catman, finally use 'pkg_create -b cyrus-sasl2-saslauthd-2.1.22' or uninstall the port.
Comment 1 Edwin Groothuis freebsd_committer freebsd_triage 2008-10-07 01:28:58 UTC 
Responsible Changed
From-To: freebsd-ports-bugs->ume

Over to maintainer (via the GNATS Auto Assign Tool)
Comment 2 dfilter service freebsd_committer freebsd_triage 2008-10-13 16:10:53 UTC 
ume         2008-10-13 15:10:44 UTC

  FreeBSD ports repository

  Modified files:
    security/cyrus-sasl2 Makefile 
    security/cyrus-sasl2/files patch-saslauthd::Makefile.in 
    security/cyrus-sasl2-saslauthd Makefile pkg-plist 
  Log:
  - Install saslauthd.mdoc to ${PREFIX}/man/man8/saslauthd.8.
  - Use ${BDB_LIB_DIR}.
  
  PR:             ports/127809
  Submitted by:   "Scot W. Hetzel" <swhetzel__at__gmail.com>
  
  Revision  Changes    Path
  1.52      +7 -5      ports/security/cyrus-sasl2-saslauthd/Makefile
  1.7       +0 -1      ports/security/cyrus-sasl2-saslauthd/pkg-plist
  1.136     +3 -1      ports/security/cyrus-sasl2/Makefile
  1.4       +1 -1      ports/security/cyrus-sasl2/files/patch-saslauthd::Makefile.in
_______________________________________________
cvs-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/cvs-all
To unsubscribe, send any mail to "cvs-all-unsubscribe@freebsd.org"
Comment 3 Hajimu UMEMOTO freebsd_committer freebsd_triage 2008-10-13 16:11:26 UTC 
State Changed
From-To: open->closed

Thanks! committed.