On Sat, May 27, 2006 at 01:17:23AM +0200, steven mestdagh wrote:
> I think you could set PKGNAME=ipython-0.7.1.1.

Done -- thanks for the feedback! I obviously missed a couple obvious
points...

> We don't install gzipped man pages.

Of course; I overlooked that. In trying to resolve this, though, I
stumbled across a weird error that I haven't gotten my head around
yet. From my Makefile:

    | [...]
    | pre-fake:
    |     @cd ${WRKSRC}/doc && gunzip *.1.gz
    | 
    | post-install:
    |     ${INSTALL_MAN} ${WRKSRC}/doc/ipython.1 ${PREFIX}/man/man1
    |     ${INSTALL_MAN} ${WRKSRC}/doc/pycolor.1 ${PREFIX}/man/man1
    |     ${INSTALL_DATA_DIR} ${DOC_DIR}
    |     .for i in ${DOC_FILE}
    |         ${INSTALL_DATA} ${WRKSRC}/doc/$i ${DOC_DIR}
    |     .endfor 
    |     ${INSTALL_DATA_DIR} ${EXAMPLE_DIR}
    |     .for i in ${EXAMPLE_FILE}
    |         ${INSTALL_DATA} ${WRKSRC}/doc/examples/$i ${EXAMPLE_DIR}
    |     .endfor 
    |     ${INSTALL_DATA_DIR} ${MANUAL_DIR}
    |     ${INSTALL_DATA} ${WRKSRC}/doc/manual/* ${MANUAL_DIR}
    | [...]

When I run make(1) (regardless of the target), I get the following
error:

    | $ sudo make; echo $?
    | "Makefile", line 39: Need an operator
    | "Makefile", line 40: Need an operator
    | Fatal errors encountered -- cannot continue
    | 1

Lines 39 and 40 are the two in my Makefile that have ${INSTALL_MAN};
commenting them out allows the build to proceed.

IPython does have a setup.py which would install the manpages, but
I've patched that behavior away as it also adds 'share/man' and
'share/man/man1' to pkg/PLIST, which causes pkg_delete(1) to
complain during uninstall:

    | $ sudo pkg_delete ipython-0.7.1
    | ipython-0.7.1: complete
    | Clean shared items: complete
    | --- ipython-0.7.1 -------------------
    | Error deleting directory /usr/local/share/man/man1: Directory not empty
    | Error deleting directory /usr/local/share/man: Directory not empty

Should pkg/PLIST be munged? Is there a better way to gunzip and
install these manpages?

/me awaits the cluestick

> PLIST is not right (try make update-plist).

Fixed.

> Maybe you could make DESCR a bit shorter?

Fixed.

> I guess it also needs NO_REGRESS, unless there is a testing procedure
> included?

No, none. Fixed.

Thanks again; playing around with the ports system has been oddly
enjoyable. I'll attach the current port with the INSTALL_MAN stuff
uncommented and the patched setup.py -- at the moment, this means no
manpages. I'm sure I'm overlooking something obvious, though...

-- 

o--------------------------{ Will Maier }--------------------------o
| jabber:[EMAIL PROTECTED] | [EMAIL PROTECTED] |
| freenode:..............lt_kije | freenode:........#madlug,#wilug |
*------------------[ BSD Unix: Live Free or Die ]------------------*

Attachment: ipython.tar.gz
Description: application/tar-gz

Reply via email to