Date: Sun, 6 Apr 2003 22:53:14 +0100 From: "Piero" <piero@poprostu.pl> To: Joe Marcus Clarke <marcus@marcuscom.com>, Piero <piero@poprostu.pl> Cc: freebsd-gnome@freebsd.org Subject: Re: port update: net/gnugadu Message-ID: <20030406215314.M63740@poprostu.pl> In-Reply-To: <1049657743.77551.30.camel@shumai.marcuscom.com> References: <20030406195554.4927c22e.piero@poprostu.pl> <1049657743.77551.30.camel@shumai.marcuscom.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On 06 Apr 2003 15:35:43 -0400, Joe Marcus Clarke wrote > On Sun, 2003-04-06 at 13:55, Piero wrote: > > > > 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) OK, thanks. > * USE_GTK: this is obsolete, do not include this (the > updated GNOME porting guide should say this [hint, hint ;-) > ]) I know, I know, this port is indeed a practical excercise to understand what I have already read in the porter's handbook. > * 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. I was looking for information and examples there, and the quote below was the first I found, so I modeled the Makefile according to it. # The logic of this section is like this: # # .if defined(WANT_GNOME) && !defined(WITHOUT_GNOME) # .for foo in ALL_GNOME_COMPONENTS # .if defined(WITH_GNOME) # HAVE_GNOME += foo # .elif (foo installed) # HAVE_GNOME += foo # .else # Print option message # .endif # .endfor # .endif Now I see, that below it all the uncommented stuff has justified dots. Maybe this also should have, though it probably would look less legible here with the dots justified to the hashes, but this is what inspirated me, and I might not be the only one. > * 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 I will take a look. > * defined(WITH_GNOME): USE_GNOME=yes 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 The thing is, you can with no problem use the port without Gnome1 installed, I was using it for a while on Fluxbox and now I use it on Gnome2. So I wouldn't like to depend this all on the libpanel. > * port-patch: Don't use Perl here. You want to define > USE_REINPLACE=yes, then use ${REINPLACE_CMD} to do your regexp > substitutions I was considering it. I kept this, as it was in the old version. Yeah, I know perl is being kicked out of the base system. I will also verify whether this patches are still needed. > * post-install: ${INSTALL_PROGRAM} usually takes care of stripping > binaries for you. It also was preserved from the old version. I just wanted to ensure that my suspicions were right. I will kick it away. Thanks for all advices, -- Piero piero@poprostu.pl
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20030406215314.M63740>