Bug 59760 - [maintainer update] Port change: zed
Summary: [maintainer update] Port change: zed
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: Sergei Kolobov
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2003-11-28 14:50 UTC by exile
Modified: 2003-12-13 00:26 UTC (History)
0 users

See Also:


Attachments
zed.diff-1.0.5_1 (2.61 KB, text/plain)
2003-11-28 14:50 UTC, exile
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description exile 2003-11-28 14:50:19 UTC
	Change in port: zed console text editor, highly configurable.
        Current version of the port has problem of applying patch-zed.h possible
        change in the original packet. Additionally FreeBSD specific fix was done
        to main.cc which remedies the problem of Zombie childs upon using the built
        in shell/make utility which uses fork().

        o Pumped the portrevision.
        o Added last modification time to Makefile

        Tested under FreeBSD-4.x branch, no idea about 5.x

Fix: # Enclosed is the diff file against the old port
How-To-Repeat: 	Current port refused to cleanly apply patches
Comment 1 Sergei Kolobov freebsd_committer freebsd_triage 2003-11-28 16:34:33 UTC
Responsible Changed
From-To: freebsd-ports-bugs->sergei

I'll handle it.
Comment 2 Sergei Kolobov freebsd_committer freebsd_triage 2003-11-29 23:16:36 UTC
Sten,

I am planning to commit the change from your PR to the ports tree once
the freeze is over. While we still have some time until that happens,
can you please look into the following port improvements:

- respect PREFIX - right now it hard-codes /usr/local
- respect CFLAGS/CXXFLAGS
- respect NOPORTDOCS
- use DOCSDIR in pkg-plist

I believe it may be easier to override the native install mechanism by
do-install target in the port's Makefile.

Please let me know if you have any questions.

Sergei
Comment 3 Sergei Kolobov freebsd_committer freebsd_triage 2003-11-29 23:17:45 UTC
State Changed
From-To: open->feedback

Ask maintainer for some additional changes.
Comment 4 exile 2003-12-03 20:16:54 UTC
Thanks for the valid remarks, the small patch below should make the port
a bit more respectful.

diff -ruN zed.sent/Makefile zed/Makefile
--- zed.sent/Makefile   Fri Nov 28 12:30:31 2003
+++ zed/Makefile        Wed Dec  3 22:16:04 2003
@@ -8,12 +8,25 @@
 
 PORTNAME=      zed
 PORTVERSION=   1.0.5
-PORTREVISION=   1
+PORTREVISION=  1
 CATEGORIES=    editors
 MASTER_SITES=  http://zed.c3po.it/
 
 MAINTAINER=    exile@chamber.ee
 COMMENT=       Simple, fast, powerful, small, highly configurable TEXT
EDITOR
+
+PREFIX=                /usr/local
+DOCSDIR=       ${PREFIX}/share/doc/${PORTNAME}
+
+MAKE_ARGS=     CFLAGS="${CFLAGS}"
+
+do-install:
+       ${INSTALL_PROGRAM} ${WRKSRC}/zed ${PREFIX}/bin/zed
+       ${INSTALL_DATA} ${WRKSRC}/cfg/zedrc ${PREFIX}/etc/zedrc
+.if !defined(NOPORTDOCS)
+       ${MKDIR} ${DOCSDIR}
+       ${INSTALL_DATA} ${WRKSRC}/zed.doc ${DOCSDIR}/zed.doc
+.endif
 
 post-install:
        @${STRIP_CMD} ${PREFIX}/bin/zed
diff -ruN zed.sent/pkg-plist zed/pkg-plist
--- zed.sent/pkg-plist  Sat Sep 21 09:19:24 2002
+++ zed/pkg-plist       Wed Dec  3 22:01:35 2003
@@ -1,4 +1,4 @@
 bin/zed
 etc/zedrc
-share/doc/zed/zed.doc
-@dirrm share/doc/zed
+%%PORTDOCS%%%%DOCSDIR%%/zed.doc
+%%PORTDOCS%%@dirrm %%DOCSDIR%%
Comment 5 Sergei Kolobov freebsd_committer freebsd_triage 2003-12-13 00:25:57 UTC
State Changed
From-To: feedback->closed

Committed, thanks!