Date: 06 Apr 2003 15:35:43 -0400 From: Joe Marcus Clarke <marcus@marcuscom.com> To: Piero <piero@poprostu.pl> Cc: freebsd-gnome@freebsd.org Subject: Re: port update: net/gnugadu Message-ID: <1049657743.77551.30.camel@shumai.marcuscom.com> In-Reply-To: <20030406195554.4927c22e.piero@poprostu.pl> References: <20030406195554.4927c22e.piero@poprostu.pl>
next in thread | previous in thread | raw e-mail | index | archive | help
--=-IpsF2p/990rNASBipMga Content-Type: text/plain Content-Transfer-Encoding: quoted-printable On Sun, 2003-04-06 at 13:55, Piero wrote: > Hi, >=20 > This is my first rewrite of a port Makefile. Can someone look at this if > this is correct? The original from CVS was broken and outdated (0.2.2 < > 1.0.0).=20 >=20 > I put comments with questions into the file where I had my doubts. I > would remove it before submitting the send-pr. It has been checked with > portlint also and it reported one warning: >=20 > WARN: "PKGNAMESUFFIX" has to appear earlier in Makefile. This is fine. >=20 > And please don't shot me right away if that's just a pitty. Okay, a few things. * USE_GNOME: gtk12 implies glib12 (you don't need both) * USE_GTK: this is obsolete, do not include this (the updated GNOME porting guide should say this [hint, hint ;-)]) * HAVE_GNOME:Mesound: (Actually, I've been told by one of the KDE people they like esound better than aRts) All dots in the conditionals should be left justified. Feel free to then indent the if, else, etc. Look at bsd.gnome.mk for an example. If you do want aRts support, you will have to depend on it somehow * HAVE_GNOME:Mlibpanel: You might want to consider modifying the PKGNAMESUFFIX here, too. Take a ride back in net/gaim's CVS for an example * defined(WITH_GNOME): USE_GNOME=3Dyes is obsolete. Never use this. You should roll this section up under the libpanel thing. Once you have libpanel, you have just about everything that's GNOME anyway. That is, you have the GNOME 1 desktop * port-patch: Don't use Perl here. You want to define USE_REINPLACE=3Dyes, then use ${REINPLACE_CMD} to do your regexp substitutions * post-install: ${INSTALL_PROGRAM} usually takes care of stripping binaries for you. You might want to do an install without this, then run `file ${PREFIX}/bin/gg` to see if it's already stripped Joe >=20 > Thanks, > --- > Piero > piero@poprostu.pl >=20 >=20 >=20 >=20 > ______________________________________________________________________ >=20 > _______________________________________________ > freebsd-gnome@freebsd.org mailing list > http://lists.freebsd.org/mailman/listinfo/freebsd-gnome > To unsubscribe, send any mail to "freebsd-gnome-unsubscribe@freebsd.org" --=20 PGP Key : http://www.marcuscom.com/pgp.asc --=-IpsF2p/990rNASBipMga Content-Type: application/pgp-signature; name=signature.asc Content-Description: This is a digitally signed message part -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.2.1 (FreeBSD) iD8DBQA+kIGPb2iPiv4Uz4cRAo9lAJ4n6I6U+q5o0DhrN0Rvyy+JEIgqcwCfaihN lot2HpC/c6JH4AyoGF2TBAg= =xl+e -----END PGP SIGNATURE----- --=-IpsF2p/990rNASBipMga--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?1049657743.77551.30.camel>