On 2015/03/18 20:58, Dmitrij D. Czarkoff wrote:
> I actually forgot that I already fixed it in openbsd-wip, and I actually
> credited you for the fix. I guess I attached outdated tarball...
Heh, no worries :)
FIrst, you will need to import txt2man. The version in github needs
NO_TEST=Yes then it is OK sthen@ to import.
> > For mdbtools, the subpackage name doesn't match the pseduo-flavour name,
> > so the automatic mechanism won't strip out the package from BUILD_PACKAGES
> > so we need to help it:
> >
> > .if ${FLAVOR:Mno_x11}
> > BUILD_PACKAGES := ${BUILD_PACKAGES:N-gmdb}
> > .endif
>
> Fixed.
>
> > > +man/mant/
> >
> > This should have been stripped by the mtree, I'm not sure why it got
> > added to PLIST, perhaps update-plist was confused :)
>
> I keep deleting it, and update-plist keeps re-adding it. It outpaces
> me.
Oh I see what it is now, I must be blind! It is mant (letter t), I thought
it was man1 (number 1).
The @comment you have now is OK, or you can rmdir it in post-install,
I am happy with either.
It looks like the pcre dependency and WANTLIB are not needed (shown as
"extra" in port-lib-depends-check)? Unless you know of a reason to keep
them, I would remove them.
Regarding committing this one, I think a bunch of add/rm is the best
way here. You can do these against cvs and send a "cvs diff -uNp",
which will also prove that the add/rm's are right.
