New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
dev-util/android-ndk: bump to 14b #4479
Conversation
Package-Manager: Portage-2.3.3, Repoman-2.3.1
Pull Request assignment Areas affected: ebuilds dev-util/android-ndk: @perfinion |
ANDROID_PATH="${ANDROID_PATH}:${ANDROID_PREFIX}/${i}" | ||
done | ||
|
||
printf '%s' \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this a fancy way of saying echo "foo"
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes. That's true. See e5d5076.
Package-Manager: Portage-2.3.5, Repoman-2.3.1
|
||
echo "SEARCH_DIRS_MASK=\"${EPREFIX}/${ANDROID_NDK_DIR}\"" > "${T}/80${PN}" || die | ||
insinto "/etc/revdep-rebuild" | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would also kill this empty line. It makes it quite confusing that you have echo and insinto in one block but the actual doins in another. But this is something that can be easily fixed while merging, so no need to push a fix.
commit 38a2cd2
Thanks for the patch! committed :D Sorry I didnt notice the PR earlier. |
Package-Manager: Portage-2.3.3, Repoman-2.3.1