pkgsrc-WIP-review archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: wip/siproxd



> The comment is very very short.
> At least ALG should be expanded.

OK

> It shouldn't start with an article either (pkglint complains about it).
> 
> > I am in particular interested in the use of PKG_SYSCONFDIR,
> > INSTALLATION_DIRS and the treatment of PLIST.
> > Also, should I better use ${LOCALBASE}/share/examples/siproxd for the
> > example config file? Should I use CONF_FILES?
> 
> Yes, definitely. You should modify the package to install the files
> in ${PREFIX}/share/examples/siproxd and use CONF_FILES to copy them to
> PKG_SYSCONFDIR automatically.
> 
> > Should I not use the original Makefile's install but rather install the
> > files myself using an explicit do-install: in my Makefile?
> 
> Usually we try to use the package's Makefile, since it is easier during
> updates (you don't have to check manually if the package would want to install
> more files).
> 
> > PS: watch out for wip/linphone which I have currently in the making...
> > literally... 
> 
> Comments for that:
> . use the inet6 option instead of USE_INET6

could you please point me to some example code?

> . BUILD_DEPENDS should come earlier (right after the COMMENT section,
>   in an own section)

is there a document that shows the correct order of potential Makefile
entries.

> . PKGMANDIR in the PLIST is unnecessary, just use "man"

but why is the substitution then part of pkgsrc? I think it is much more
transparent to use PKGMANDIR in the PLIST than to have man silently
rewritten.


> . The --with-html-dir configure argument should point to
>   ${PREFIX}/share/doc/linphone

no html in the path?

> . share/gnome/help/linphone are empty, so perhaps just remove them

I must admit I am still puzzled on what linphone actually installs and why.

>   in a post-install step instead of adding them to the PLIST (they
>   won't get create this way if you don't @exec ${MKDIR} them).
> 
> I haven't tried compiling either of them yet.

please do :-)


-- 
Georg Schwarz    http://home.pages.de/~schwarz/
 georg.schwarz%freenet.de@localhost  +49 178 8545053



-------------------------------------------------------
SF.Net email is sponsored by:
Tame your development challenges with Apache's Geronimo App Server. Download
it for free - -and be entered to win a 42" plasma tv or your very own
Sony(tm)PSP.  Click here to play: http://sourceforge.net/geronimo.php
_______________________________________________
pkgsrc-wip-review mailing list
pkgsrc-wip-review%lists.sourceforge.net@localhost
https://lists.sourceforge.net/lists/listinfo/pkgsrc-wip-review



Home | Main Index | Thread Index | Old Index