Skip to content
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

Renamed subdirectories in Android SKD's build-tools #12447

Merged
merged 1 commit into from
Jan 18, 2016

Conversation

yacinehmito
Copy link
Contributor

Nix unzips the different components of the Android SDK one by one.
It followed the directory structure of complete packages released for
mainstream OS but the names of the directories in build-tools doesn't
match those.
As a result, some programs assuming the usual directory structure and
naming conventions broke (in my case it is a gradle plugin).
This is a fix. It may introduce a regression if some programs rely on
the current behavior.

Nix unzips the different components of the Android SDK one by one.
It followed the directory structure of complete packages released for
mainstream OS but the names of the directories in build-tools doesn't
match those.
As a result, some programs assuming the usual directory structure and
naming conventions broke (in my case it is a gradle plugin).
This is a fix. It may introduce a regression if some programs rely on
the current behavior.
@mention-bot
Copy link

By analyzing the blame information on this pull request, we identified @MP2E, @svanderburg and @taku0 to be potential reviewers

@nckx
Copy link
Member

nckx commented Jan 18, 2016

If it's what upstream uses, we should too. And Travis says nothing breaks, so... thanks!

Don't see any maintainers; merging.

nckx added a commit that referenced this pull request Jan 18, 2016
Renamed subdirectories in Android SKD's build-tools
@nckx nckx merged commit 5db2412 into NixOS:master Jan 18, 2016
@Janik-Haag Janik-Haag added the 12. first-time contribution This PR is the author's first one; please be gentle! label Jun 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
12. first-time contribution This PR is the author's first one; please be gentle!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants