Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 374279 - dev-util/android-ndk-5c: env.d entry incorrectly contains spaces in PATH
Summary: dev-util/android-ndk-5c: env.d entry incorrectly contains spaces in PATH
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Doug Goldstein (RETIRED)
URL: http://developer.android.com/sdk/ndk/...
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-07-06 22:12 UTC by Markus Rathgeb
Modified: 2012-08-13 17:24 UTC (History)
0 users

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

Attachments
ebuild patch (android-ndk-5c.ebuild.diff,623 bytes, patch)
2011-08-28 04:28 UTC, Joe Pelkey 		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 Markus Rathgeb 2011-07-06 22:12:29 UTC 
The ebuild created a file for /etc/env.d at "src_install()" state with the folling lines:
        echo "PATH=\"${EPREFIX}/${ANDROID_NDK_DIR}:" \
                "${EPREFIX}/${ANDROID_TC_ARM_EABI}/bin/:" \
                "${EPREFIX}/${ANDROID_TC_ANDROID_EABI}/bin/\"" \
                > "${T}/80${PN}"  || die

The content of the created file contains annoying white spaces.
E.g.:
PATH="/opt/android-ndk: /opt/android-ndk/toolchains/arm-eabi-4.4.0/prebuilt/linux-x86/bin/: /opt/android-ndk/toolchains/arm-linux-androideabi-4.4.3/prebuilt/linux-x86/bin/"

The whitespaces should be removed from the PATH environment.

Reproducible: Always
Comment 1 Markus Rathgeb 2011-07-09 17:35:33 UTC 
Not new line, spaces.
Bugzilla insert the line breaks because of the spaces.
Comment 2 Joe Pelkey 2011-08-28 04:28:45 UTC 
Created attachment 284849 [details, diff]
ebuild patch

I confirmed this bug when it caused the build of app-text/linuxdoc-tools-0.9.21_p11 to fail.  I attached an ebuild patch that fixes the problem.
Comment 3 Doug Goldstein (RETIRED) gentoo-dev 2012-08-13 17:24:36 UTC 
Should be fixed in the 8b version. Thanks for the fix Joe.

commit message:
Version bump to the latest Android NDK. Fixes for the space in the PATH from Joe Pelkey <pelkeyj@gmail.com>. bug #374279. bump bug #392377.