[arch-projects] [devtools][PATCH] checkpkg: Fix soname check
The bsdtar options were in the incorrect order and objdump couldn't find the files. Signed-off-by: Eric Bélanger <snowmaniscool@gmail.com> --- checkpkg.in | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/checkpkg.in b/checkpkg.in index ef46399..02c6543 100644 --- a/checkpkg.in +++ b/checkpkg.in @@ -73,9 +73,9 @@ for _pkgname in "${pkgname[@]}"; do if diff "$TEMPDIR/filelist-$_pkgname"{-old,} | grep '\.so' &>/dev/null; then mkdir -p "$TEMPDIR/pkg" - bsdtar -C "$TEMPDIR" xf ../"$pkgfile" #> /dev/null + bsdtar x -C "$TEMPDIR" -f "$pkgfile" #> /dev/null diff "$TEMPDIR/filelist-$_pkgname-old" "$TEMPDIR/filelist-$_pkgname" | awk '/>.*\.so/{$1 = ""; print $0}' | while read i; do - echo "${i}: " "$(objdump -p "$i" | grep SONAME)" + echo "${i}: " "$(objdump -p "$TEMPDIR/$i" | grep SONAME)" done else msg "No soname differences for $_pkgname." -- 1.8.4.1
On Wed, Oct 23, 2013 at 11:24:45PM -0400, Eric Bélanger wrote:
The bsdtar options were in the incorrect order and objdump couldn't find the files.
Signed-off-by: Eric Bélanger <snowmaniscool@gmail.com> --- checkpkg.in | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/checkpkg.in b/checkpkg.in index ef46399..02c6543 100644 --- a/checkpkg.in +++ b/checkpkg.in @@ -73,9 +73,9 @@ for _pkgname in "${pkgname[@]}"; do
if diff "$TEMPDIR/filelist-$_pkgname"{-old,} | grep '\.so' &>/dev/null; then mkdir -p "$TEMPDIR/pkg" - bsdtar -C "$TEMPDIR" xf ../"$pkgfile" #> /dev/null + bsdtar x -C "$TEMPDIR" -f "$pkgfile" #> /dev/null
Problem here doesn't seem to be order of flags, but the missing '-' before 'xf'.
diff "$TEMPDIR/filelist-$_pkgname-old" "$TEMPDIR/filelist-$_pkgname" | awk '/>.*\.so/{$1 = ""; print $0}' | while read i; do - echo "${i}: " "$(objdump -p "$i" | grep SONAME)" + echo "${i}: " "$(objdump -p "$TEMPDIR/$i" | grep SONAME)" done else msg "No soname differences for $_pkgname." -- 1.8.4.1
On Thu, Oct 24, 2013 at 10:05 AM, Dave Reisner <d@falconindy.com> wrote:
The bsdtar options were in the incorrect order and objdump couldn't find
On Wed, Oct 23, 2013 at 11:24:45PM -0400, Eric Bélanger wrote: the files.
Signed-off-by: Eric Bélanger <snowmaniscool@gmail.com> --- checkpkg.in | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/checkpkg.in b/checkpkg.in index ef46399..02c6543 100644 --- a/checkpkg.in +++ b/checkpkg.in @@ -73,9 +73,9 @@ for _pkgname in "${pkgname[@]}"; do
if diff "$TEMPDIR/filelist-$_pkgname"{-old,} | grep '\.so'
&>/dev/null; then
mkdir -p "$TEMPDIR/pkg" - bsdtar -C "$TEMPDIR" xf ../"$pkgfile" #> /dev/null + bsdtar x -C "$TEMPDIR" -f "$pkgfile" #> /dev/null
Problem here doesn't seem to be order of flags, but the missing '-' before 'xf'.
I confirm adding a '-' in front of xf fixes it. Is that important enough to submit a new patch?
diff "$TEMPDIR/filelist-$_pkgname-old"
"$TEMPDIR/filelist-$_pkgname" | awk '/>.*\.so/{$1 = ""; print $0}' | while read i; do
- echo "${i}: " "$(objdump -p "$i" | grep SONAME)" + echo "${i}: " "$(objdump -p "$TEMPDIR/$i" | grep SONAME)" done else msg "No soname differences for $_pkgname." -- 1.8.4.1
participants (2)
-
Dave Reisner
-
Eric Bélanger