On 13 September 2011 09:05, b. f. <bf1...@googlemail.com> wrote: >> This problem has been going on for a number of years, and as far as I >> can see there isn't a proper and elegant solution. >> >> A fix is at [1], which should solve the problem, and I'm going to >> tinderbox test and commit it if no-one complains. > > As Kozlov said earlier, that's a bit ugly. I think it would be better > to avoid attempting to build the module in the first place, by simply > adding something like: > > .if !exists(/usr/bin/ypcat) # the world with NO_NIS > ${REINPLACE_CMD} -e 's|rpcsvc/yp_prot.h|${NONEXISTENT}|' \ > ${PATCH_WRKSRC}/setup.py > .endif > > to the pre-patch target, or something like: > > .if !exists(/usr/bin/ypcat) # the world with NO_NIS > ${REINPLACE_CMD} -e '\|disabled_module_list =|s|]|, "nis"&|' \ > ${PATCH_WRKSRC}/setup.py > .endif
Good point, and thanks for the idea, even if it did take me ten minutes to work out how the hell your REINPLACE line worked ;) I've simplified it a little. Is it better now? It works in tinderbox so far.... http://www.bayofrum.net/~crees/patches/python-nis_failed-fix-bffix.diff Chris _______________________________________________ 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"