252342 – media-sound/mpd-0.14_beta3: Incorrect operator precedence in src_install function

Bug 252342 - media-sound/mpd-0.14_beta3: Incorrect operator precedence in src_install function

Summary: media-sound/mpd-0.14_beta3: Incorrect operator precedence in src_install func...

Status:	RESOLVED FIXED

Alias:	None

Product:	Gentoo Linux
Classification:	Unclassified
Component:	New packages (show other bugs)
Hardware:	All Linux

Importance:	High normal (vote)
Assignee:	Christoph Mende (RETIRED)

URL:
Whiteboard:
Keywords:

Depends on:
Blocks:

Reported:	2008-12-24 01:49 UTC by Anna Gabutero
Modified:	2008-12-25 10:21 UTC (History)
CC List:	1 user (show)

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

Attachments
Proposed fix (mpd-0.14_beta3.ebuild.patch,742 bytes, patch) 2008-12-24 01:51 UTC, Anna Gabutero	Details \| Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Anna Gabutero 2008-12-24 01:49:59 UTC

The src_install function checks USE flags to determine whether to perform certain sed operations.  Due to operator precedence, this causes the ebuild to fail with the message "dosed failed" if either USE flag is disabled.

Reproducible: Always

Steps to Reproduce:
Emerge media-sound/mpd-0.14_beta3 with either the unicode or alsa USE flag disabled.
Actual Results:  
Package fails to install with the message "dosed failed".

Expected Results:  
Package should install correctly.

Comment 1 Anna Gabutero 2008-12-24 01:51:46 UTC

Created attachment 176267 [details, diff]
Proposed fix

Comment 2 Wormo (RETIRED) gentoo-dev

2008-12-25 06:28:34 UTC

Thanks for the report, and for the fix! Assigning to maintainers.

Comment 3 Christoph Mende (RETIRED) gentoo-dev

2008-12-25 10:21:57 UTC

thanks, changed it to if use x; instead