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

Add .nomedia file to Android assets #10047

Merged
merged 2 commits into from Jun 18, 2020
Merged

Conversation

rubenwardy
Copy link
Member

@rubenwardy rubenwardy commented Jun 18, 2020

.nomedia files prevent images from appearing in the phone's gallery app and other places.

This file was present in the old way of extracting/building assets

To do

  • Check this works recursively
  • Testing

How to test

Use Minetest, check gallery app

Gallery apps may cache the pictures, and won't recheck .nomedia. It also expects .nomedia to be the first file, which may not happen if the folder is already indexed (crazy assumption?!) I needed to delete the Minetest folder and reextract the assets for this to work.

@rubenwardy rubenwardy added @ Build CMake, build scripts, official builds, compiler and linker errors Android Bugfix 🐛 PRs that fix a bug labels Jun 18, 2020
@rubenwardy rubenwardy added this to the 5.3.0 milestone Jun 18, 2020
@rubenwardy rubenwardy added the Trivial The change is a trivial bug fix, documentation or maintenance change, as per the Git Guidelines label Jun 18, 2020
@@ -1043,7 +1043,7 @@ void TouchScreenGUI::applyJoystickStatus()
if (i == 4 && !m_joystick_triggers_special1)
continue;

SEvent translated;
SEvent translated{};
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

😂

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure why this doesn't just have an empty diff?

@MoNTE48
Copy link
Contributor

MoNTE48 commented Jun 18, 2020

Does it work on Windows? If not, MultiCraft creates this file from Java.

This file was present in the old way of extracting/building assets

Wow, impossible! Where?

@rubenwardy
Copy link
Member Author

Not sure, it may be done in the Java. I can check. I'm not sure which way is better

Yes, this does work on Windows :)

@MoNTE48
Copy link
Contributor

MoNTE48 commented Jun 18, 2020

When worked - leave it as is.

@rubenwardy rubenwardy merged commit 469e8d8 into minetest:master Jun 18, 2020
@rubenwardy rubenwardy deleted the addnomedia branch June 18, 2020 17:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Android Bugfix 🐛 PRs that fix a bug @ Build CMake, build scripts, official builds, compiler and linker errors Trivial The change is a trivial bug fix, documentation or maintenance change, as per the Git Guidelines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants