Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 23 Feb 2004 07:50:22 -0800 (PST)
From:      Ulrich Spoerlein <q@uni.de>
To:        freebsd-ports-bugs@FreeBSD.org
Subject:   Re: ports/63222: [patch] Remove useless use of cat 5/5
Message-ID:  <200402231550.i1NFoMit001313@freefall.freebsd.org>

next in thread | raw e-mail | index | archive | help
The following reply was made to PR ports/63222; it has been noted by GNATS.

From: Ulrich Spoerlein <q@uni.de>
To: Michael Nottebrock <michaelnottebrock@gmx.net>
Cc:  
Subject: Re: ports/63222: [patch] Remove useless use of cat 5/5
Date: Mon, 23 Feb 2004 16:46:01 +0100

 On Sun, 22.02.2004 at 15:38:18 +0100, Michael Nottebrock wrote:
 > Impressive waste of time, way to go.
 
 Just making sure, new porters don't see bad examples, so they don't
 repeat them.
 
 Ideally I would place some default %%foo%% expansions into bsd.port.mk,
 that way you could have the substituted pkg-message in the package by
 default. But I'm pretty sure people are against this.=20
 
 > > =A0post-install:
 > > =A0=A0=A0=A0=A0=A0=A0=A0@ ${MKDIR} ${PREFIX}/share/postgresql ;\
 > > -=A0=A0=A0=A0=A0=A0=A0${CAT} ${FILESDIR}/post-install-notes ${PKGMESSAG=
 E} |\
 > > -=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0${SED} "s|/usr/local|${PR=
 EFIX}|g" |\
 > > +=A0=A0=A0=A0=A0=A0=A0${SED} "s|/usr/local|${PREFIX}|g" ${FILESDIR}/pos=
 t-install-notes \
 > > +=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0${PKGMESSAGE} |\
 > > =A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0tee ${PREFIX}/share/pos=
 tgresql/post-install-notes
 >=20
 > I'd bet that "fix" is actually slower than the original.
 
 At least on my system, you lose this bet:
 % ll gst.patch pam.list
 -rw-r--r--  1 q  wheel  1196 Feb  8 21:41 gst.patch
 -rw-r--r--  1 q  wheel  5521 Feb 17 13:37 pam.list
 % time (sed s/foo/bar/ gst.patch pam.list >/dev/null)
 0.00s user 0.01s system 70% cpu 0.011 total
 % time (sed s/foo/bar/ gst.patch pam.list >/dev/null)
 0.01s user 0.00s system 86% cpu 0.009 total
 % time (sed s/foo/bar/ gst.patch pam.list >/dev/null)
 0.00s user 0.00s system 0% cpu 0.009 total
 % time (sed s/foo/bar/ gst.patch pam.list >/dev/null)
 0.01s user 0.00s system 97% cpu 0.008 total
 % time (sed s/foo/bar/ gst.patch pam.list >/dev/null)
 0.00s user 0.01s system 357% cpu 0.002 total
                         ^^^
                         wtf?
 
 % time (cat gst.patch pam.list|sed s/foo/bar/ >/dev/null)
 0.00s user 0.01s system 49% cpu 0.016 total
 % time (cat gst.patch pam.list|sed s/foo/bar/ >/dev/null)
 0.00s user 0.01s system 47% cpu 0.016 total
 % time (cat gst.patch pam.list|sed s/foo/bar/ >/dev/null)
 0.00s user 0.01s system 49% cpu 0.016 total
 % time (cat gst.patch pam.list|sed s/foo/bar/ >/dev/null)
 0.00s user 0.01s system 53% cpu 0.015 total
 % time (cat gst.patch pam.list|sed s/foo/bar/ >/dev/null)
 0.00s user 0.01s system 56% cpu 0.014 total
                         ^^
 CPU usage is lower on overall, which indicates increased
 latency due to I/O overhead(??).
 
 Ulrich Sp=F6rlein
 --=20
 PGP Key ID: F0DB9F44				Get it while it's hot!
 PGP Fingerprint: F1CE D062 0CA9 ADE3 349B  2FE8 980A C6B5 F0DB 9F44
 I abhor a system designed for the "user", if that word is a coded
 pejorative meaning "stupid and unsophisticated".	-- Ken Thompson



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