From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <mike@fly.osdn.org.ua>
X-Spam-Checker-Version: SpamAssassin 3.2.5 (2008-06-10) on sa.int.altlinux.org
X-Spam-Level: 
X-Spam-Status: No, score=-1.8 required=5.0 tests=AWL,BAYES_00 autolearn=ham
	version=3.2.5
Date: Sun, 31 May 2009 20:21:30 +0300
From: Michael Shigorin <mike@osdn.org.ua>
To: sisyphus@lists.altlinux.org
Message-ID: <20090531172130.GV12156@osdn.org.ua>
Mail-Followup-To: sisyphus@lists.altlinux.org
Mime-Version: 1.0
Content-Type: text/plain; charset=koi8-r
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
User-Agent: Mutt/1.4.2.1i
Subject: [sisyphus] [JT] Fwd: Re: packages: dmenu/dmenu.spec (NEW) - initial
X-BeenThere: sisyphus@lists.altlinux.org
X-Mailman-Version: 2.1.12
Precedence: list
Reply-To: shigorin@gmail.com,
	ALT Linux Sisyphus discussions <sisyphus@lists.altlinux.org>
List-Id: ALT Linux Sisyphus discussions <sisyphus.lists.altlinux.org>
List-Unsubscribe: <https://lists.altlinux.org/mailman/options/sisyphus>,
	<mailto:sisyphus-request@lists.altlinux.org?subject=unsubscribe>
List-Archive: <http://lists.altlinux.org/pipermail/sisyphus>
List-Post: <mailto:sisyphus@lists.altlinux.org>
List-Help: <mailto:sisyphus-request@lists.altlinux.org?subject=help>
List-Subscribe: <https://lists.altlinux.org/mailman/listinfo/sisyphus>,
	<mailto:sisyphus-request@lists.altlinux.org?subject=subscribe>
X-List-Received-Date: Sun, 31 May 2009 17:21:45 -0000
Archived-At: <http://lore.altlinux.org/sisyphus/20090531172130.GV12156@osdn.org.ua/>
List-Archive: <http://lore.altlinux.org/sisyphus/>
List-Post: <mailto:sisyphus@altlinux.ru>

1) интересно, чем именно "much more secure"
2) а что за filter-out?  в федориных патчах видел, у нас нет

----- Forwarded message from Przemyslaw Iskra <sparky/pld-linux.org> -----

Date: Sun, 31 May 2009 19:04:13 +0200
From: Przemyslaw Iskra <sparky/pld-linux.org>
To: pld-devel-en/lists.pld-linux.org
Subject: Re: packages: dmenu/dmenu.spec (NEW) - initial

On Sun, May 31, 2009 at 06:45:28PM +0200, uzsolt wrote:
> Author: uzsolt                       Date: Sun May 31 16:45:28 2009 GMT
> Module: packages                      Tag: HEAD
> ---- Log message:
> - initial

> +%prep
> +%setup -q
> +sed -i "s/^PREFIX.*/PREFIX=%{_prefix}/" config.mk
> +sed -i "s/^\(CFLAGS.*\)-Os\(.*\)/\1 \2 %{rpmcflags}/" config.mk
> +sed -i "s/^\(LDFLAGS.*\)/\1 %{rpmldflags}/" config.mk

You shouldn't be defining these at %prep stage, doing so in %build is
much more secure. And there must be much a simpler way than using sed
to do it. You could pass options in %__make invocation, or append them
at the end of config.mk file:

%build
cat << 'EOF' >> config.mk
PREFIX = %{_prefix}
CFLAGS := %{rpmcflags} $(filter-out -Os,$(CFLAGS))
LDFLAGS = %{rpmldflags}
EOF

%{__make}


BTW, appropriate sed BR is missing.

-- 
 ____  Sparky{PI] -- Przemyslaw _  ___  _  _  ........... LANG...Pl..Ca..Es..En
/____) ___  ___  _ _ || Iskra  |  | _ \| |  | : WWW........ppcrcd.pld-linux.org
\____\| -_)'___| ||^'||//\\// <   |  _/| |  | : JID......sparky<at>jabberes.org
(____/||   (_-_|_||  ||\\ ||   |_ |_|  |_| _| : Mail....sparky<at>pld-linux.org
_______________________________________________
pld-devel-en mailing list
pld-devel-en/lists.pld-linux.org
http://lists.pld-linux.org/mailman/listinfo/pld-devel-en

----- End forwarded message -----

-- 
 ---- WBR, Michael Shigorin <mike@altlinux.ru>
  ------ Linux.Kiev http://www.linux.kiev.ua/