Re: PYDISTUTILS_AUTOPLIST, stage, man pages

2013-10-18 Thread Marcus von Appen 

Andriy Gapon :


on 18/10/2013 09:44 Marcus von Appen said the following:

Can you give the following change to Mk/bsd.python.mk a try:

Index: Mk/bsd.python.mk
===
--- Mk/bsd.python.mk(revision 330721)
+++ Mk/bsd.python.mk(working copy)
@@ -528,7 +528,9 @@
${SED} '/^\.$$/d' > ${WRKDIR}/.localmtree
@${ECHO_CMD} "${_RELSITELIBDIR}" >> ${WRKDIR}/.localmtree
@${ECHO_CMD} "${_RELLIBDIR}" >> ${WRKDIR}/.localmtree
-   @${SED} 's|^${PREFIX}/||' ${_PYTHONPKGLIST} | ${SORT} >> ${TMPPLIST}
+   @${SED} -e 's|^${PREFIX}/||' \
+   -e 's|^\(man/man[0-9]\)/\(.*.[0-9]\)|\1/\2${MANEXT}|' \
+   ${_PYTHONPKGLIST} | ${SORT} >> ${TMPPLIST}
@${SED} -e 's|^${PREFIX}/\(.*\)/\(.*\)|\1|' ${_PYTHONPKGLIST} | \
${AWK} '{ num = split($$0, a, "/"); res=""; \
for(i = 1; i <= num; ++i) { \

It adds the .gz to every man/manX/*.X, if it is defined (depending on
the NO_MANCOMPRESS setting).


Woo-hoo, thanks a lot!  The patch worked perfectly for me.
From pkg info -l py27-obnam-1.5:
/usr/local/man/man1/obnam-benchmark.1.gz
/usr/local/man/man1/obnam-viewprof.1.gz
/usr/local/man/man1/obnam.1.gz


Excellent! I'll commit it later on.

Cheers
Marcus

___
freebsd-python@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-python
To unsubscribe, send any mail to "freebsd-python-unsubscr...@freebsd.org"

Re: PYDISTUTILS_AUTOPLIST, stage, man pages

2013-10-18 Thread John Hein 
Marcus von Appen wrote at 09:39 +0200 on Oct 18, 2013:
 > Andriy Gapon :
 > 
 > > on 18/10/2013 09:44 Marcus von Appen said the following:
 > >> Can you give the following change to Mk/bsd.python.mk a try:
 > >>
 > >> Index: Mk/bsd.python.mk
 > >> ===
 > >> --- Mk/bsd.python.mk  (revision 330721)
 > >> +++ Mk/bsd.python.mk  (working copy)
 > >> @@ -528,7 +528,9 @@
 > >>   ${SED} '/^\.$$/d' > ${WRKDIR}/.localmtree
 > >>   @${ECHO_CMD} "${_RELSITELIBDIR}" >> ${WRKDIR}/.localmtree
 > >>   @${ECHO_CMD} "${_RELLIBDIR}" >> ${WRKDIR}/.localmtree
 > >> - @${SED} 's|^${PREFIX}/||' ${_PYTHONPKGLIST} | ${SORT} >> ${TMPPLIST}
 > >> + @${SED} -e 's|^${PREFIX}/||' \
 > >> + -e 's|^\(man/man[0-9]\)/\(.*.[0-9]\)|\1/\2${MANEXT}|' \
 > >> + ${_PYTHONPKGLIST} | ${SORT} >> ${TMPPLIST}
 > >>   @${SED} -e 's|^${PREFIX}/\(.*\)/\(.*\)|\1|' ${_PYTHONPKGLIST} | \
 > >>   ${AWK} '{ num = split($$0, a, "/"); res=""; \
 > >>   for(i = 1; i <= num; ++i) { \
 > >>
 > >> It adds the .gz to every man/manX/*.X, if it is defined (depending on
 > >> the NO_MANCOMPRESS setting).
 > >
 > > Woo-hoo, thanks a lot!  The patch worked perfectly for me.
 > > From pkg info -l py27-obnam-1.5:
 > > /usr/local/man/man1/obnam-benchmark.1.gz
 > > /usr/local/man/man1/obnam-viewprof.1.gz
 > > /usr/local/man/man1/obnam.1.gz
 > 
 > Excellent! I'll commit it later on.


Maybe you should escape the '.' before the [0-9] (like below).

Also if ${_PYTHONPKGLIST} already has entries with, then the patch
above will append another ${MANEXT}.  Maybe adding '$' (also below)
will be better.

Is it worth trying to handle man{l,n}?  They are defined in
BSD.local.dist, but I don't know of any ports that use them anymore?

Index: Mk/bsd.python.mk
===
--- Mk/bsd.python.mk(revision 330721)
+++ Mk/bsd.python.mk(working copy)
@@ -528,7 +528,9 @@
${SED} '/^\.$$/d' > ${WRKDIR}/.localmtree
@${ECHO_CMD} "${_RELSITELIBDIR}" >> ${WRKDIR}/.localmtree
@${ECHO_CMD} "${_RELLIBDIR}" >> ${WRKDIR}/.localmtree
-   @${SED} 's|^${PREFIX}/||' ${_PYTHONPKGLIST} | ${SORT} >> ${TMPPLIST}
+   @${SED} -e 's|^${PREFIX}/||' \
+   -e 's|^\(man/man[0-9]\)/\(.*\.[0-9]\)$|\1/\2${MANEXT}|' \
+   ${_PYTHONPKGLIST} | ${SORT} >> ${TMPPLIST}
@${SED} -e 's|^${PREFIX}/\(.*\)/\(.*\)|\1|' ${_PYTHONPKGLIST} | \
${AWK} '{ num = split($$0, a, "/"); res=""; \
for(i = 1; i <= num; ++i) { \
___
freebsd-python@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-python
To unsubscribe, send any mail to "freebsd-python-unsubscr...@freebsd.org"

Re: PYDISTUTILS_AUTOPLIST, stage, man pages

2013-10-18 Thread Marcus von Appen 
On, Fri Oct 18, 2013, John Hein wrote:

> Marcus von Appen wrote at 09:39 +0200 on Oct 18, 2013:
>  > Andriy Gapon :
>  >
>  > > on 18/10/2013 09:44 Marcus von Appen said the following:
>  > >> Can you give the following change to Mk/bsd.python.mk a try:
>  > >>
>  > >> Index: Mk/bsd.python.mk
>  > >> ===
>  > >> --- Mk/bsd.python.mk(revision 330721)
>  > >> +++ Mk/bsd.python.mk(working copy)
>  > >> @@ -528,7 +528,9 @@
>  > >> ${SED} '/^\.$$/d' > ${WRKDIR}/.localmtree
>  > >> @${ECHO_CMD} "${_RELSITELIBDIR}" >> ${WRKDIR}/.localmtree
>  > >> @${ECHO_CMD} "${_RELLIBDIR}" >> ${WRKDIR}/.localmtree
>  > >> -   @${SED} 's|^${PREFIX}/||' ${_PYTHONPKGLIST} | ${SORT} >> 
> ${TMPPLIST}
>  > >> +   @${SED} -e 's|^${PREFIX}/||' \
>  > >> +   -e 's|^\(man/man[0-9]\)/\(.*.[0-9]\)|\1/\2${MANEXT}|' \
>  > >> +   ${_PYTHONPKGLIST} | ${SORT} >> ${TMPPLIST}
>  > >> @${SED} -e 's|^${PREFIX}/\(.*\)/\(.*\)|\1|' ${_PYTHONPKGLIST} | 
> \
>  > >> ${AWK} '{ num = split($$0, a, "/"); res=""; \
>  > >> for(i = 1; i <= num; ++i) { \
>  > >>
>  > >> It adds the .gz to every man/manX/*.X, if it is defined (depending on
>  > >> the NO_MANCOMPRESS setting).
>  > >
>  > > Woo-hoo, thanks a lot!  The patch worked perfectly for me.
>  > > From pkg info -l py27-obnam-1.5:
>  > > /usr/local/man/man1/obnam-benchmark.1.gz
>  > > /usr/local/man/man1/obnam-viewprof.1.gz
>  > > /usr/local/man/man1/obnam.1.gz
>  >
>  > Excellent! I'll commit it later on.
>
>
> Maybe you should escape the '.' before the [0-9] (like below).
>
> Also if ${_PYTHONPKGLIST} already has entries with, then the patch
> above will append another ${MANEXT}.  Maybe adding '$' (also below)
> will be better.

Thanks, I committed it that way.

>
> Is it worth trying to handle man{l,n}?  They are defined in
> BSD.local.dist, but I don't know of any ports that use them anymore?

I do not know of any - as long as there are no specific requests for
those, I'd ignore those both directories.

Cheers
Marcus


pgp5DB_HBDIXL.pgp
Description: PGP signature

Re: ports/182952: [PATCH] lang/python wrong cflags in python-config on systems where cc = clang

2013-10-18 Thread koobs 
Synopsis: [PATCH] lang/python wrong cflags in python-config on systems where cc 
= clang

Responsible-Changed-From-To: freebsd-python->koobs
Responsible-Changed-By: koobs
Responsible-Changed-When: Sat Oct 19 05:12:07 UTC 2013
Responsible-Changed-Why: 
I'll take it.

http://www.freebsd.org/cgi/query-pr.cgi?pr=182952
___
freebsd-python@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-python
To unsubscribe, send any mail to "freebsd-python-unsubscr...@freebsd.org"