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

1941 Global Command Decision map folder has -master folder #12993

Open
TheDog-GH opened this issue Nov 12, 2024 · 6 comments
Open

1941 Global Command Decision map folder has -master folder #12993

TheDog-GH opened this issue Nov 12, 2024 · 6 comments
Labels
Problem A problem, bug, defect - something to fix

Comments

@TheDog-GH
Copy link
Contributor

@asvitkine @DanVanAtta @frigoref @beelee1 @RogerCooper

I have just updated 1941 Global Command Decision to v190 on GitHub

I thought I had done what I usually do, but for some reason the map now downloads with a folder called
1941_global_command_decision-master

inside 1941_global_command_decision folder

The Zip also has the same, please advise on how to correct this and remove the 1941_global_command_decision-master folder

GitHub map link
https://github.com/triplea-maps/1941_global_command_decision

@TheDog-GH TheDog-GH added the Problem A problem, bug, defect - something to fix label Nov 12, 2024
@beelee1
Copy link
Contributor

beelee1 commented Nov 12, 2024

yea I noticed that a couple months ago. Idky it does it :)

@panther2
Copy link
Contributor

This seems to be caused by the TripleA-installation-process:
https://forums.triplea-game.org/post/65331

@TheDog-GH
Copy link
Contributor Author

TheDog-GH commented Nov 17, 2024

Thanks to Panther for pointing me in the right direction.

This is the version that causes the additional folder with a -master addition
https://github.com/triplea-game/triplea/releases/tag/2.7.14825

@DanVanAtta
When you have time, please take a look.

@frigoref
Copy link
Member

frigoref commented Nov 17, 2024

@TheDog-GH, @beelee1, @panther2 Let me try to summarize what the requested change of functionality to the current behavior is:
Map zip files that contain only one subfolder (e.g. a folder of name pattern '*-master') are supposed to be unzipped AND (this is the new part) the contained subfolder level should be removed, i.e., the files contained in this subfolder should be moved up on level and the (now empty) subfolder on this level should be removed.

With the link from @TheDog-GH I found the code place which seems to have taken care of this before which got deleted:
image

PR 13002 (Unzip without master subfolder) created and merged.

Please provide feedback.

PS: Please try to express clearly at the beginning of each issue the as-is, to-be, the difference and reasoning for change to make it easier especially for the technical guys to understand the what and why for a change.

@TheDog-GH
Copy link
Contributor Author

for the to-be
There are 2 variants and I can only comment on one, the most recent. Yes to
the files contained in this subfolder should be moved up on level and the (now empty) subfolder on this level should be removed

.
The 2nd older variant I cannot comment on, as I have not seen it, so don't know what it looks like.

@panther2
Copy link
Contributor

PR 13002 (Unzip without master subfolder) created and merged.

Please provide feedback.

TripleA 2.7.14900 installs the downloaded map as expected again. Thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Problem A problem, bug, defect - something to fix
Projects
None yet
Development

No branches or pull requests

4 participants