2013/5/7 Stuart Henderson <[email protected]> > On 2013/05/06 14:51, Vadim Zhukov wrote: > > Index: bsd.port.mk.5 > > =================================================================== > > RCS file: /cvs/src/share/man/man5/bsd.port.mk.5,v > > retrieving revision 1.373 > > diff -u -p -r1.373 bsd.port.mk.5 > > --- bsd.port.mk.5 25 Mar 2013 13:13:13 -0000 1.373 > > +++ bsd.port.mk.5 4 May 2013 11:33:49 -0000 > > @@ -1852,6 +1852,16 @@ tedious. > > .It Ev MODGNU_CONFIG_GUESS_DIRS > > If a port uses config.guess outside WRKSRC, the directories > > containing the other copies must be set here. > > +.It Ev MODPERL_ADJ_FILES > > +List of executable scripts under > > +.Ev WRKSRC > > +which should have shebangs adjusted to safe form like > > +.Dq #!/usr/bin/perl . > > +.It Ev MODPERL_BIN_ADJ > > +Shell fragment to patch the interpreter path in executable > > scripts. > > +If names of files to be patched are known, they better to be > > listed in > > +.Ev MODPERL_ADJ_FILES > > +instead. > > .It Ev MODPERL_BUILD_TARGET > > Normal content of > > .Ar do-build > > > > > > ping. > > Here's alternative wording, some of it borrowed from port-modules(5) > where it talks about the same thing. >
Looks like much better than mine, thanks! One minor nit below, though: > Index: bsd.port.mk.5 > =================================================================== > RCS file: /cvs/src/share/man/man5/bsd.port.mk.5,v > retrieving revision 1.373 > diff -u -p -r1.373 bsd.port.mk.5 > --- bsd.port.mk.5 25 Mar 2013 13:13:13 -0000 1.373 > +++ bsd.port.mk.5 6 May 2013 20:07:27 -0000 > @@ -1852,6 +1852,19 @@ tedious. > .It Ev MODGNU_CONFIG_GUESS_DIRS > If a port uses config.guess outside WRKSRC, the directories > containing the other copies must be set here. > +.It Ev MODPERL_ADJ_FILES > > +If any files have a perl shebang line which needs to be replaced > +with > +.Dq #!/usr/bin/perl , > +list them in > +.Ev MODPERL_ADJ_FILES . > +These are prefixed with WRKSRC and replaced automatically > +at the end of > > > > > +.Ar pre-configure . > The word "prefixed" is about file names, and "replaced" is about shebang lines in files. So mixing them in one sentence creates some sort of confusion, at least for me. Maybe something like that, if I'm not mistaken? If any files have a Perl shebang line, which needs to be replaced with .Dq #!/usr/bin/perl , list them in .Ev MODPERL_ADJ_FILES . File paths here should be relative to .Ev WRKSRC . These files are patched automatically at the end of .Ar pre-configure . > > +.It Ev MODPERL_BIN_ADJ > +Shell fragment to patch the Perl interpreter path in executable scripts. > +Used by > +.Ev MODPERL_ADJ_FILES . > .It Ev MODPERL_BUILD_TARGET > Normal content of > .Ar do-build > > -- WBR, Vadim Zhukov
