On Tue, Jan 16, 2007 at 11:05:23PM +0100, VMiklos <[EMAIL PROTECTED]> wrote: > On Tue, Jan 16, 2007 at 10:35:42PM +0100, Wael Nasreddine <[EMAIL PROTECTED]> > wrote: > > Well of course, it's only a sed command, needs adjusting, I agree that it's > > ugly, but I did this way myself to guarantee that the Substitution/adding > > works... > > > > Anyway I'll try to adjust it (even that I'm not a sed expert). > > what about something like this? > > if [ "`grep -c sha1sums= FrugalBuild`" -gt 1 ]; then > # handle the problem about complex sha1sums like-archspecific > ones: > # bail out with an error or so > fi > > if grep sha1sums= FrugalBuild; then > Fmessage "updating \$sha1sums() in the FrugalBuild" > if grep -q '^sha1sums.*)' FrugalBuild; then > sed -i "/^sha1sums.*)/d" FrugalBuild > else > sed -i "/^sha1sums/,/)/d" FrugalBuild > fi > makepkg -dg >>FrugalBuild > fi > > this way we would not need such ### FOO ### blocks imho > That's a good idea, would you change it or should I update the patch ?? BTW I like the way the sha1sums are printed in the patch more than it was, two sha1sums on the same line then the next right under the 1st one, better than each one on a new line (take a good look at the patch)
-- Wael Nasreddine http://wael.nasreddine.com PGP: 1024D/C8DD18A2 06F6 1622 4BC8 4CEB D724 DE12 5565 3945 C8DD 18A2 .: An infinite number of monkeys typing into GNU emacs, would never make a good program. (L. Torvalds 1995) :. _______________________________________________ Frugalware-devel mailing list [email protected] http://frugalware.org/mailman/listinfo/frugalware-devel
