> What does BSD sed currently do? I am assuming it leaves a backup file(s) but
> can you clarify the numbers and names of these?
On Mac OS X (Man page is from FreeBSD, so I believe it would be the same there):
sed -i -e 's/foo/bar/' filename
will create a backup file named filename-e
sed -i -e 's/foo/bar' -e '/asd/dsa/' filename
will give an error message because
sed -i 's/foo/bar' filename will give an error message
sed -i '' 's/foo/bar/' filename will work without creating a backup
file, but fails on GNU/sed
It's really annoying that BSD sed doesn't like sed -i'' and GNU/sed
doesn't like sed -i ''
This patch changes the way that sed inplace is run in makepkg when
running in versionpkg mode. Previous, sed would run with no backup
file, called in the GNU way. It seems there is no cross platform
method of calling sed inplace with no backup (BSD/sed uses sed -i '',
GNU/sed uses sed -i'' or simply sed -i).
Instead, this patch creates a backup file (.pkgsave) and removes it
after the succesful completion of the sed operation. This means the
user should not see the backup file, unless something went run (full
hard drive?) in which case they don't lose the PKGBUILD as they would
have before.
Signed-off-by: Kevin Barry <barryk gmail com>
---
scripts/makepkg.sh.in | 5 +++--
1 files changed, 3 insertions(+), 2 deletions(-)
diff --git a/scripts/makepkg.sh.in b/scripts/makepkg.sh.in
index e10c345..8ddf88f 100644
--- a/scripts/makepkg.sh.in
+++ b/scripts/makepkg.sh.in
@@ -1199,8 +1199,9 @@ devel_update() {
if [ -n "$newpkgver" ]; then
if [ "$newpkgver" != "$pkgver" ]; then
if [ -f "./$BUILDSCRIPT" ]; then
- sed -i "s/^pkgver=[^ ]*/pkgver=$newpkgver/" "./$BUILDSCRIPT"
- sed -i "s/^pkgrel=[^ ]*/pkgrel=1/" "./$BUILDSCRIPT"
+ sed -i.pkgsave -e "s/^pkgver=[^ ]*/pkgver=$newpkgver/" \
+ -e "s/^pkgrel=[^ ]*/pkgrel=1/" "./$BUILDSCRIPT" &&
+ rm -f "./${BUILDSCRIPT}.pkgsave"
source "$BUILDSCRIPT"
fi
fi
--
1.6.1.2
_______________________________________________
pacman-dev mailing list
[email protected]
http://www.archlinux.org/mailman/listinfo/pacman-dev
