Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 395957 - net-dns/avahi-0.6.30-r2[-python] fails in src_install
Summary: net-dns/avahi-0.6.30-r2[-python] fails in src_install
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Maxim Koltsov (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-12-25 03:05 UTC by Ulrich Müller
Modified: 2011-12-25 10:48 UTC (History)
3 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
build.log (build.log,84.99 KB, text/plain)
2011-12-25 03:05 UTC, Ulrich Müller
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Ulrich Müller gentoo-dev 2011-12-25 03:05:15 UTC
Created attachment 296901 [details]
build.log

The patch from bug 383773 is not correct: With USE="-python" the ebuild doesn't install /usr/bin/avahi-discover. Therefore python_convert_shebangs fails because its target doesn't exist.

Same issue for 0.6.29-r1.
Comment 1 Maxim Koltsov (RETIRED) gentoo-dev 2011-12-25 08:57:22 UTC
Fixed.
Sorry for breaking packages.
Comment 2 Ulrich Müller gentoo-dev 2011-12-25 10:38:40 UTC
(In reply to comment #1)
> Fixed.
> Sorry for breaking packages.

It still fails with USE="-python":

  ERROR: net-dns/avahi-0.6.30-r2 failed (install phase):
    (no error message)
  
  Call stack:
      ebuild.sh, line  85:  Called src_install
    environment, line 5811:  Called die
  The specific snippet of code:
        use python && python_convert_shebangs 2 "${ED}"usr/bin/avahi-discover || die;


In bash, && and || have equal precedence, so they're executed with left associativity. So the above won't work.

Suggested fix:

    if use python; then
        python_convert_shebangs 2 "${ED}"usr/bin/avahi-discover || die
    fi
Comment 3 Maxim Koltsov (RETIRED) gentoo-dev 2011-12-25 10:48:43 UTC
This is really bad day :(
Hope it's fixed finally.