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

lib3mf-2.3.2-source-with-submodules.zip extracts itself into the wrong folder #391

Open
lvk88 opened this issue Sep 6, 2024 · 6 comments
Milestone

Comments

@lvk88
Copy link

lvk88 commented Sep 6, 2024

Issue

Downloading and extracting lib3mf-2.3.2-source-with-submodules.zip from the releases page will put all the contents of the zip file in the same folder as the zip file itself.

This is inconsistent with how the zip file for the source code is compressed. This file extracts itself into a subfolder, which, in my opinion is the correct behavior.

Steps to reproduce

wget https://github.com/3MFConsortium/lib3mf/releases/download/v2.3.2/lib3mf-2.3.2-source-with-submodules.zip
unzip lib3mf-2.3.2-source-with-submodules.zip
ls -l

The output of ls will be

total 43536
drwxrwxr-x 4 lvk88 lvk88     4096 Sep  6 20:14 Autogenerated
drwxrwxr-x 2 lvk88 lvk88     4096 Sep  6 20:14 AutomaticComponentToolkit
drwxrwxr-x 2 lvk88 lvk88     4096 Sep  6 20:14 CI
drwxrwxr-x 2 lvk88 lvk88     4096 Sep  6 20:14 cmake
-rw-r--r-- 1 lvk88 lvk88    11606 Jun 18 07:22 CMakeLists.txt
-rw-r--r-- 1 lvk88 lvk88     3672 Jun 18 07:22 CONTRIBUTING.md
drwxrwxr-x 4 lvk88 lvk88     4096 Sep  6 20:14 Documentation
drwxrwxr-x 6 lvk88 lvk88     4096 Sep  6 20:14 Include
-rw-rw-r-- 1 lvk88 lvk88 44490341 Jul  6 18:37 lib3mf-2.3.2-source-with-submodules.zip
-rw-r--r-- 1 lvk88 lvk88      311 Jun 18 07:22 lib3mf.pc.in
-rw-r--r-- 1 lvk88 lvk88     9499 Jun 18 07:22 lib3mf_version_update.py
drwxrwxr-x 8 lvk88 lvk88     4096 Sep  6 20:14 Libraries
-rw-r--r-- 1 lvk88 lvk88     1292 Jun 18 07:22 LICENSE
-rw-r--r-- 1 lvk88 lvk88     3236 Jun 18 07:22 README.md
drwxrwxr-x 4 lvk88 lvk88     4096 Sep  6 20:14 SDK 
drwxrwxr-x 5 lvk88 lvk88     4096 Sep  6 20:14 Source
drwxrwxr-x 8 lvk88 lvk88     4096 Sep  6 20:14 submodules
drwxrwxr-x 6 lvk88 lvk88     4096 Sep  6 20:14 Tests

Compare this to the standard source code zip:

total 37576
drwxrwxr-x 14 lvk88 lvk88     4096 Jul  3 14:14 lib3mf-2.3.2                                                                                                                                                       
-rw-rw-r--  1 lvk88 lvk88 38470801 Sep  6 20:15 v2.3.2.zip

Detected this issue while working on cnr-isti-vclab/meshlab/pull/1521

@vijaiaeroastro
Copy link
Collaborator

@lvk88 You can simply unzip with a directory name

unzip lib3mf-2.3.2-source-with-submodules.zip -d lib3mf-2.3.2

You can give any directory name with -d option

@lvk88
Copy link
Author

lvk88 commented Sep 6, 2024

Thanks @vijaiaeroastro for the quick answer.

3MFConsortium could also zip the file in a way that is consistent with the other archives on the releases page. 😈

Edit:

but bad jokes aside, why I see this as an issue is that in the project where lib3mf 2.2.0 have been used so far, I recently tried to update to 2.3.2 by changing the name of the zip that is to be downloaded from the releases page. Sadly, this didn't work, because 2.2.0 was packed in a different way than lib3mf-2.3.2-source-with-submodules.zip . It's not a huge issue, though. I thought that it might be worth pointing out.

@vijaiaeroastro
Copy link
Collaborator

@vijaiaeroastro Already fixed. Check and see if it works as expected.

@vijaiaeroastro
Copy link
Collaborator

Thanks @vijaiaeroastro for the quick answer.

3MFConsortium could also zip the file in a way that is consistent with the other archives on the releases page. 😈

Edit:

but bad jokes aside, why I see this as an issue is that in the project where lib3mf 2.2.0 have been used so far, I recently tried to update to 2.3.2 by changing the name of the zip that is to be downloaded from the releases page. Sadly, this didn't work, because 2.2.0 was packed in a different way than lib3mf-2.3.2-source-with-submodules.zip . It's not a huge issue, though. I thought that it might be worth pointing out.

This artifact was created for internal purposes. I see that it is actually useful for lib3mf users too. So, we will be more consistent in future. Thanks for your report. Much appreciated :)

@vijaiaeroastro vijaiaeroastro added this to the 2.4.0 milestone Sep 6, 2024
@lvk88
Copy link
Author

lvk88 commented Sep 6, 2024

@vijaiaeroastro thanks a lot for the quick fix!

@vijaiaeroastro
Copy link
Collaborator

@lvk88 No issues. I use MeshLab quite a lot in my day-to-day work. I would be happy to assist in getting lib3mf working in MeshLab. Feel free to create any issues / queries.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants