<p>Hi Kester,</p>
<p>Some comments on the code, below.</p>
<p>Em 28/07/2011 10:55, <<a href="mailto:khabermann@users.sourceforge.net">khabermann@users.sourceforge.net</a>> escreveu:<br>
><br>
> Revision: 15227<br>
>          <a href="http://gar.svn.sourceforge.net/gar/?rev=15227&view=rev">http://gar.svn.sourceforge.net/gar/?rev=15227&view=rev</a><br>
> Author:   khabermann<br>
> Date:     2011-07-28 09:55:15 +0000 (Thu, 28 Jul 2011)<br>
><br>
> Log Message:<br>
> -----------<br>
> dsh: created package. libdshconfig: created package</p>
<p>It's better for reviews and change tracking if you keep one commit per package.</p>
<p>><br>
> Added Paths:<br>
> -----------<br>
>    csw/mgar/pkg/dsh/<br>
>    csw/mgar/pkg/dsh/Makefile<br>
>    csw/mgar/pkg/dsh/branches/<br>
>    csw/mgar/pkg/dsh/tags/<br>
>    csw/mgar/pkg/dsh/trunk/<br>
>    csw/mgar/pkg/dsh/trunk/Makefile<br>
>    csw/mgar/pkg/dsh/trunk/checksums<br>
>    csw/mgar/pkg/dsh/trunk/files/<br>
>    csw/mgar/pkg/libdshconfig/<br>
>    csw/mgar/pkg/libdshconfig/Makefile<br>
>    csw/mgar/pkg/libdshconfig/branches/<br>
>    csw/mgar/pkg/libdshconfig/tags/<br>
>    csw/mgar/pkg/libdshconfig/trunk/<br>
>    csw/mgar/pkg/libdshconfig/trunk/Makefile<br>
>    csw/mgar/pkg/libdshconfig/trunk/checksums<br>
>    csw/mgar/pkg/libdshconfig/trunk/files/<br>
><br>
> Copied: csw/mgar/pkg/dsh/Makefile (from rev 15060, csw/mgar/pkg/template/Makefile)<br>
> ===================================================================<br>
> --- csw/mgar/pkg/dsh/Makefile                           (rev 0)<br>
> +++ csw/mgar/pkg/dsh/Makefile   2011-07-28 09:55:15 UTC (rev 15227)<br>
> @@ -0,0 +1,15 @@<br>
> +# vim: ft=make ts=4 sw=4 noet<br>
> +<br>
> +default:<br>
> +       @echo "You are in the pkg/ directory."<br>
> +<br>
> +%:<br>
> +       $(MAKE) -C trunk $*<br>
> +<br>
> +paranoid-%:<br>
> +       $(MAKE) -C trunk $* || exit 2<br>
> +<br>
> +export BUILDLOG ?= $(shell pwd)/buildlog.txt<br>
> +<br>
> +report-%:<br>
> +       $(MAKE) -C trunk $* || echo "   *** make $* in $$i failed ***" >> $(BUILDLOG)<br>
><br>
><br>
> Property changes on: csw/mgar/pkg/dsh/trunk<br>
> ___________________________________________________________________<br>
> Added: svn:ignore<br>
>   + cookies<br>
> download<br>
> work<br>
><br>
><br>
> Added: svn:externals<br>
>   + gar <a href="https://gar.svn.sourceforge.net/svnroot/gar/csw/mgar/gar/v2">https://gar.svn.sourceforge.net/svnroot/gar/csw/mgar/gar/v2</a><br>
><br>
><br>
> Added: csw/mgar/pkg/dsh/trunk/Makefile<br>
> ===================================================================<br>
> --- csw/mgar/pkg/dsh/trunk/Makefile                             (rev 0)<br>
> +++ csw/mgar/pkg/dsh/trunk/Makefile     2011-07-28 09:55:15 UTC (rev 15227)<br>
> @@ -0,0 +1,36 @@<br>
> +# $Id$<br>
> +# TODO (release-critical prefixed with !, non release-critical with *)<br>
> +#<br>
> +NAME = dsh<br>
> +VERSION = 0.25.9<br>
> +GARTYPE = v2<br>
> +CATEGORIES = devel<br>
> +RUNTIME_DEP_PKGS_CSWdsh = CSWlibdshconfig<br>
> +BUILD_DEP_PKGS = CSWlibdshconfig<br>
> +<br>
> +DESCRIPTION = Distributed shell. Runs command through rsh or ssh on a cluster of machines.<br>
> +<br>
> +MASTER_SITES = <a href="http://www.netfort.gr.jp/~dancer/software/downloads/dsh-0.25.9.tar.gz">http://www.netfort.gr.jp/~dancer/software/downloads/dsh-0.25.9.tar.gz</a><br>
> +DISTFILES  = $(DISTNAME).tar.gz<br>
> +<br>
> +# File name regex to get notifications about upstream software releases<br>
> +# NOTE: Use this only if the automatic regex creation<br>
> +#       does not work for your package<br>
> +# UFILES_REGEX = $(NAME)-(\d+(?:\.\d+)*).tar.gz</p>
<p>You can remove the unused lines.</p>
<p>> +# If the url used to check for software update is different of MASTER_SITES, then<br>
> +# uncomment the next line. Otherwise it is set by default to the value of MASTER_SITES<br>
> +# UPSTREAM_MASTER_SITES =<br>
> +<br>
> +# for libintl_gettext, libintl_textdomain, libintl_bindtextdomain<br>
> +LIBS += -lintl<br>
> +export LIBS</p>
<p>It's better to let gar handle this by setting EXTRA_LIBS (iirc). Ignition doesn't work in this case, you can make a note why the LIBS variable is exported directly. </p>
<p>> +CONFIGURE_ARGS = $(DIRPATHS)<br>
> +<br>
> +TEST_SCRIPTS =<br>
> +<br>
> +TEST_TARGET =<br>
> +<br>
> +include gar/<a href="http://category.mk">category.mk</a><br>
> +<br>
><br>
><br>
> Property changes on: csw/mgar/pkg/dsh/trunk/Makefile<br>
> ___________________________________________________________________<br>
> Added: svn:keywords<br>
>   + Id<br>
><br>
> Added: csw/mgar/pkg/dsh/trunk/checksums<br>
> ===================================================================<br>
> --- csw/mgar/pkg/dsh/trunk/checksums                            (rev 0)<br>
> +++ csw/mgar/pkg/dsh/trunk/checksums    2011-07-28 09:55:15 UTC (rev 15227)<br>
> @@ -0,0 +1 @@<br>
> +60734780242172fca9e68d223654292d  dsh-0.25.9.tar.gz<br>
><br>
> Copied: csw/mgar/pkg/libdshconfig/Makefile (from rev 15060, csw/mgar/pkg/template/Makefile)<br>
> ===================================================================<br>
> --- csw/mgar/pkg/libdshconfig/Makefile                          (rev 0)<br>
> +++ csw/mgar/pkg/libdshconfig/Makefile  2011-07-28 09:55:15 UTC (rev 15227)<br>
> @@ -0,0 +1,15 @@<br>
> +# vim: ft=make ts=4 sw=4 noet<br>
> +<br>
> +default:<br>
> +       @echo "You are in the pkg/ directory."<br>
> +<br>
> +%:<br>
> +       $(MAKE) -C trunk $*<br>
> +<br>
> +paranoid-%:<br>
> +       $(MAKE) -C trunk $* || exit 2<br>
> +<br>
> +export BUILDLOG ?= $(shell pwd)/buildlog.txt<br>
> +<br>
> +report-%:<br>
> +       $(MAKE) -C trunk $* || echo "   *** make $* in $$i failed ***" >> $(BUILDLOG)<br>
><br>
><br>
> Property changes on: csw/mgar/pkg/libdshconfig/trunk<br>
> ___________________________________________________________________<br>
> Added: svn:ignore<br>
>   + cookies<br>
> download<br>
> work<br>
><br>
><br>
> Added: svn:externals<br>
>   + gar <a href="https://gar.svn.sourceforge.net/svnroot/gar/csw/mgar/gar/v2">https://gar.svn.sourceforge.net/svnroot/gar/csw/mgar/gar/v2</a><br>
><br>
><br>
> Added: csw/mgar/pkg/libdshconfig/trunk/Makefile<br>
> ===================================================================<br>
> --- csw/mgar/pkg/libdshconfig/trunk/Makefile                            (rev 0)<br>
> +++ csw/mgar/pkg/libdshconfig/trunk/Makefile    2011-07-28 09:55:15 UTC (rev 15227)<br>
> @@ -0,0 +1,27 @@<br>
> +# $Id$<br>
> +# TODO (release-critical prefixed with !, non release-critical with *)<br>
> +#<br>
> +NAME = libdshconfig<br>
> +VERSION = 0.20.9<br>
> +GARTYPE = v2<br>
> +CATEGORIES = devel<br>
> +<br>
> +DESCRIPTION = Library for parsing dsh-style configuration files.<br>
> +<br>
> +MASTER_SITES = <a href="http://www.netfort.gr.jp/~dancer/software/downloads/libdshconfig-0.20.9.tar.gz">http://www.netfort.gr.jp/~dancer/software/downloads/libdshconfig-0.20.9.tar.gz</a><br>
> +<br>
> +DISTFILES  = $(DISTNAME).tar.gz<br>
> +<br>
> +# File name regex to get notifications about upstream software releases<br>
> +# NOTE: Use this only if the automatic regex creation<br>
> +#       does not work for your package<br>
> +# UFILES_REGEX = $(NAME)-(\d+(?:\.\d+)*).tar.gz<br>
> +<br>
> +# If the url used to check for software update is different of MASTER_SITES, then<br>
> +# uncomment the next line. Otherwise it is set by default to the value of MASTER_SITES<br>
> +# UPSTREAM_MASTER_SITES =</p>
<p>Same comment about unused lines.</p>
<p>> +CONFIGURE_ARGS = $(DIRPATHS)<br>
> +<br>
> +include gar/<a href="http://category.mk">category.mk</a><br>
> +<br>
><br>
><br>
> Property changes on: csw/mgar/pkg/libdshconfig/trunk/Makefile<br>
> ___________________________________________________________________<br>
> Added: svn:keywords<br>
>   + Id<br>
><br>
> Added: csw/mgar/pkg/libdshconfig/trunk/checksums<br>
> ===================================================================<br>
> --- csw/mgar/pkg/libdshconfig/trunk/checksums                           (rev 0)<br>
> +++ csw/mgar/pkg/libdshconfig/trunk/checksums   2011-07-28 09:55:15 UTC (rev 15227)<br>
> @@ -0,0 +1 @@<br>
> +c3fabfae1782c17ee989373c2f6a73c1  libdshconfig-0.20.9.tar.gz<br>
><br>
><br>
> This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.<br>
> _______________________________________________<br>
> devel mailing list<br>
> <a href="mailto:devel@lists.opencsw.org">devel@lists.opencsw.org</a><br>
> <a href="https://lists.opencsw.org/mailman/listinfo/devel">https://lists.opencsw.org/mailman/listinfo/devel</a><br>
</p>