<div class="gmail_quote">Hi Carsten,</div><div class="gmail_quote"><br></div><div class="gmail_quote">Overall, it looks like you've submitted the change without testing if the build works and produces the output you want. If you can, try to build and review first, and submit second, so that you don't submit things like silly typos.</div>
<div class="gmail_quote"><br></div><div class="gmail_quote">More comments below.</div><div class="gmail_quote"><br></div><div class="gmail_quote">2012/9/21 <span dir="ltr"><<a href="mailto:cgrzemba@users.sourceforge.net" target="_blank">cgrzemba@users.sourceforge.net</a>></span><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Revision: 19258<br>
<a href="http://gar.svn.sourceforge.net/gar/?rev=19258&view=rev" target="_blank">http://gar.svn.sourceforge.net/gar/?rev=19258&view=rev</a><br>
Author: cgrzemba<br>
Date: 2012-09-21 10:53:43 +0000 (Fri, 21 Sep 2012)<br>
Log Message:<br>
-----------<br>
libsoup2/trunk: rework packaging<br>
<br>
Modified Paths:<br>
--------------<br>
csw/mgar/pkg/libsoup2/trunk/Makefile<br>
<br>
Modified: csw/mgar/pkg/libsoup2/trunk/Makefile<br>
===================================================================<br>
--- csw/mgar/pkg/libsoup2/trunk/Makefile 2012-09-21 10:21:43 UTC (rev 19257)<br>
+++ csw/mgar/pkg/libsoup2/trunk/Makefile 2012-09-21 10:53:43 UTC (rev 19258)<br>
@@ -10,34 +10,56 @@<br>
MASTER_SITES = $(GNOME_MIRROR)<br>
DISTFILES = $(NAME)-$(VERSION).tar.bz2<br>
<br>
+PACKAGING_PLATFORMS = Solaris10-sparc Solaris10-i386<br>
<br>
# We define upstream file regex so we can be notifed of new upstream software release<br>
UFILES_REGEX = -(\d+(?:\.\d+)*).tar.bz2<br></blockquote><div><br></div><div>The above line is not necessary any more, there's a function that constructs these regexes on the fly.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
-PACKAGES = CSWlibsoup2<br>
-PACKAGES += CSWlibsoup2dev<br>
-SPKG_DESC_CSWlibsoup2 = HTTP client/server library for GNOME<br>
-SPKG_DESC_CSWlibsoup2dev = HTTP client/server library for GNOME, dev files<br>
+PACKAGES += CSWlibsoup2-4-1<br>
+CATALOGNAME_CSWlibsoup2-4-1 = libsoup2_4_1<br>
+PKGFILES_CSWlibsoup2-4-1 += $(call baseisadirs,$(libdir),libsoup-2\.4\.so\.1\.3\.0)<br>
+PKGFILES_CSWlibsoup2-4-1 += $(call baseisadirs,$(libdir),libsoup-2\.4\.so\.1(\.\d+)*)<br></blockquote><div><br></div><div>You only need one line of the two lines above. Checkpkg shows both, and it's your job to choose which one you really need. I'd say, throw away the first one.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
+SPKG_DESC_CSWlibsoup2-4-1 += $(DESCRIPTION), libsoup-2.4.so.1<br>
+RUNTIME_DEP_PKGS_CSWlibsoup2-4-1 += CSWlibgthread2-0-0<br>
+RUNTIME_DEP_PKGS_CSWlibsoup2-4-1 += CSWlibgmodule2-0-0<br>
+RUNTIME_DEP_PKGS_CSWlibsoup2-4-1 += CSWlibz1<br>
+RUNTIME_DEP_PKGS_CSWlibsoup2-4-1 += CSWlibintl8<br>
+RUNTIME_DEP_PKGS_CSWlibsoup2-4-1 += CSWlibgobject2-0-0<br>
+RUNTIME_DEP_PKGS_CSWlibsoup2-4-1 += CSWlibglib2-0-0<br>
+RUNTIME_DEP_PKGS_CSWlibsoup2-4-1 += CSWlibxml2-2<br>
+RUNTIME_DEP_PKGS_CSWlibsoup2-4-1 += CSWlibgio2-0-0<br>
+OBSOLTED_BY_CSWlibsoup2-4-1 = CSWlibsoup<br></blockquote><div><br></div><div>Typo, should be OBSOLETED_BY.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
-RUNTIME_DEP_PKGS_CSWlibsoup2 = CSWgconf2<br>
-RUNTIME_DEP_PKGS_CSWlibsoup2 += CSWgcrypt<br>
-RUNTIME_DEP_PKGS_CSWlibsoup2 += CSWggettextrt<br>
-RUNTIME_DEP_PKGS_CSWlibsoup2 += CSWglib2<br>
-RUNTIME_DEP_PKGS_CSWlibsoup2 += CSWgnutls<br>
-RUNTIME_DEP_PKGS_CSWlibsoup2 += CSWgpgerr<br>
-RUNTIME_DEP_PKGS_CSWlibsoup2 += CSWlibproxy<br>
-RUNTIME_DEP_PKGS_CSWlibsoup2 += CSWlibxml2<br>
-RUNTIME_DEP_PKGS_CSWlibsoup2 += CSWsqlite3rt<br>
+PACKAGES += CSWlibsoup-gnome2-4-1<br>
+CATALOGNAME_CSWlibsoup-gnome2-4-1 = libsoup_gnome2_4_1<br></blockquote><div><br></div><div>The CATALOGNAME line is not necessary, gar guesses that dashes should be substituted by underscores.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
+PKGFILES_CSWlibsoup-gnome2-4-1 += $(call baseisadirs,$(libdir),libsoup-gnome-2\.4\.so\.1\.3\.0)<br>
+PKGFILES_CSWlibsoup-gnome2-4-1 += $(call baseisadirs,$(libdir),libsoup-gnome-2\.4\.so\.1(\.\d+)*)<br></blockquote><div><br></div><div>Same comment, only one line needed.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
+SPKG_DESC_CSWlibsoup-gnome2-4-1 += $(DESCRIPTION), libsoup-gnome-2.4.so.1<br>
+RUNTIME_DEP_PKGS_CSWlibsoup-gnome2-4-1 += CSWlibgthread2-0-0<br>
+RUNTIME_DEP_PKGS_CSWlibsoup-gnome2-4-1 += CSWlibsqlite3-0<br>
+RUNTIME_DEP_PKGS_CSWlibsoup-gnome2-4-1 += CSWlibgmodule2-0-0<br>
+RUNTIME_DEP_PKGS_CSWlibsoup-gnome2-4-1 += CSWlibz1<br>
+RUNTIME_DEP_PKGS_CSWlibsoup-gnome2-4-1 += CSWlibintl8<br>
+RUNTIME_DEP_PKGS_CSWlibsoup-gnome2-4-1 += CSWlibgnome-keyring0<br>
+RUNTIME_DEP_PKGS_CSWlibsoup-gnome2-4-1 += CSWlibsoup2-4-1<br>
+RUNTIME_DEP_PKGS_CSWlibsoup-gnome2-4-1 += CSWlibgobject2-0-0<br>
+RUNTIME_DEP_PKGS_CSWlibsoup-gnome2-4-1 += CSWlibgio2-0-0<br>
+RUNTIME_DEP_PKGS_CSWlibsoup-gnome2-4-1 += CSWlibxml2-2<br>
+RUNTIME_DEP_PKGS_CSWlibsoup-gnome2-4-1 += CSWlibglib2-0-0<br>
+OBSOLTED_BY_CSWlibsoup-gnome2-4-1 = CSWlibsoup<br></blockquote><div><br></div><div>The same typo.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
-RUNTIME_DEP_PKGS_CSWlibsoup2dev += CSWlibsoup2<br>
+PACKAGES += CSWlibsoup2-dev<br>
+CATALOGNAME_CSWlibsoup2-dev = libsoup2_dev<br></blockquote><div><br></div><div>Not necessary.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
+SPKG_DESC_CSWlibsoup2-dev += $(DESCRIPTION), development files<br>
+# PKGFILES_CSWlibsoup2-dev += /opt/csw/lib/<a href="http://libsoup-2.4.so" target="_blank">libsoup-2.4.so</a><br>
+PKGFILES_CSWlibsoup2-dev += $(PKGFILES_DEVEL)<br>
+RUNTIME_DEP_PKGS_CSWlibsoup2-dev += CSWlibsoup2-4-1<br>
+RUNTIME_DEP_PKGS_CSWlibsoup2-dev += CSWlibsoup-gnome2-4-1<br>
+OBSOLETED_BY_CSWlibsoup2-dev = CSWlibsoupdevel<br>
+CATALOGNAME_CSWlibsoupdevel = libsoup_devel_stub<br>
<br>
-<br>
-PKGFILES_CSWlibsoup2dev = /opt/csw/lib/pkgconfig/.*<br>
-PKGFILES_CSWlibsoup2dev += /opt/csw/include/.*<br>
-<br>
-<br>
-EXTRA_INC = /opt/csw/X11/include<br>
-EXTRA_LIB = /opt/csw/X11/lib<br>
+# EXTRA_INC = /opt/csw/X11/include<br>
+# EXTRA_LIB = /opt/csw/X11/lib<br>
EXTRA_PKG_CONFIG_DIRS = $(prefix)/X11/lib<br>
STRIP_LIBTOOL = 1<br>
<br>
<br>
This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.<br>
<br>
_______________________________________________<br>
devel mailing list<br>
<a href="mailto:devel@lists.opencsw.org" target="_blank">devel@lists.opencsw.org</a><br>
<a href="https://lists.opencsw.org/mailman/listinfo/devel" target="_blank">https://lists.opencsw.org/mailman/listinfo/devel</a><br>
</blockquote></div><br>