Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 9 Mar 2002 11:20:45 +0100
From:      "Simon 'corecode' Schubert" <corecode@corecode.ath.cx>
To:        Sue Blake <sue@welearn.com.au>
Cc:        alane@geeksrus.net, ports@FreeBSD.ORG
Subject:   Re: wannabee porter in the poo
Message-ID:  <20020309112045.34354145.corecode@corecode.ath.cx>
In-Reply-To: <20020309191436.B84857@welearn.com.au>
References:  <20020309003638.A84857@welearn.com.au> <200203081414.g28EEHk80378@wwweasel.geeksrus.net> <20020309191436.B84857@welearn.com.au>

next in thread | previous in thread | raw e-mail | index | archive | help
--=.RVvj3uY6Vg.o0l
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit

On Sat, 9 Mar 2002 19:14:36 +1100 Sue Blake <sue@welearn.com.au> wrote:

> Thanks for the many tips, I'm now well on the way.
> After abandoning man pages for now and emphasising the supplied docs
> instead, just one more question should see me through.
> 
> On Fri, Mar 08, 2002 at 09:14:16AM -0500, Alan E wrote:
> > On Friday 08 March 2002 08:36, you wrote:
> > 
> > > Converting makefile:
> > > The makefile that comes with the sources is in DOS text
> > > format (CR/LF), and the notes warn that it needs to be
> > > changed over before compiling to avoid errors.
> > > It produces the most astonishing patch you've ever seen :-)
> > > How should I really be tackling this one?
> > >
> > 
> > IMO, in the pre-patch target:
> > 
> > pre-patch:
> > 	${MV} ${WRKSRC}/Makefile ${WRKSRC}/Makefile.dos
> > 	${TR} -d '\r' < ${WRKSRC}/Makefile.dos > ${WRKSRC}/Makefile
> > 
> > The reason I say pre-patch, not post-patch, is that you will likely need to 
> > apply a patch to the Makefile as well, and wouldn't you rather work on it 
> > after that conversion is over and done with?
> 
> Surprise surprise :-) I have the same file format problem with the text
> files I'm installing with ${INSTALL_MAN}. Should I convert them
> in the work dir first, and if so, where in the makefile should
> that command go?

i'd say pre-patch.
something like

pre-patch:
..for f in ${DOSFILES}
	@${PERL} -i.dos -e 's/\r//' ${WRKSRC}/$f
..endfor

or, if you got only dos files and lots of them:

pre-patch:
	@${FIND} ${WRKSRC} -type f | ${XARGS} \
		${PERL} -i.dos -e 's/\r//'


-- 
/"\   http://corecode.ath.cx/
\ /
 \     ASCII Ribbon Campaign
/ \  Against HTML Mail and News

--=.RVvj3uY6Vg.o0l
Content-Type: application/pgp-signature

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.0.6 (FreeBSD)

iD8DBQE8ieIAr5S+dk6z85oRAqjDAKDR0/Okjd9pjtFAViylWLnUz6599wCgmVAn
PVKISsjpiJmtYZPK5z2RUEk=
=SJC4
-----END PGP SIGNATURE-----

--=.RVvj3uY6Vg.o0l--


To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe freebsd-ports" in the body of the message




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20020309112045.34354145.corecode>