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

Several plugin ZIPs using QGIS Plugin CI have duplicated files inside #111

Closed
kannes opened this issue Feb 4, 2022 · 9 comments · Fixed by #112
Closed

Several plugin ZIPs using QGIS Plugin CI have duplicated files inside #111

kannes opened this issue Feb 4, 2022 · 9 comments · Fixed by #112

Comments

@kannes
Copy link

kannes commented Feb 4, 2022

For some reason I downloaded all QGIS plugins and extracted them. I noticed an issue and it looks like QGIS Plugin CI is a common demoninator. If it is not the cause, sorry! :)

resources.py is included twice in 5 plugins:

They all seem to use QGIS Plugin CI so I wonder if this is caused by some bug in it or misconfiguration at the plugin CIs?

@lbartoletti
Copy link

cc @Guts (for QompliGIS, I think I already spotted this issue for LSCI)

@3nids
Copy link
Member

3nids commented Feb 4, 2022

Thanks for investigating this!
For swiss locator (downloaded from the GH repo release asset), I see resources.qrc and resources_rc.py at the root?
Are you saying we should avoid shipping the .qrc file?

@Gustry
Copy link
Collaborator

Gustry commented Feb 4, 2022

No, he means the resources.py file is duplicated in the ZIP : https://github.com/opengisch/qgis-swiss-locator/releases/download/1.4.6/swiss_locator.1.4.6.zip

@Gustry
Copy link
Collaborator

Gustry commented Feb 4, 2022

When it's extracted, one file is deleted by the OS, logical ...

@kannes
Copy link
Author

kannes commented Feb 4, 2022

I should add, this causes no issues when installing the plugins via QGIS's plugin installer. I guess it will just overwrite the duplicate.

This is mostly a cosmetic thing but it might be worth fixing for the sake of clean packaging. I only noticed it because I unpacked the ZIPs with standard archive tools.

@3nids
Copy link
Member

3nids commented Feb 4, 2022

I guess the issue is when there is already one _rc.py file in the sources, which should not happen normally.

@3nids
Copy link
Member

3nids commented Feb 4, 2022

image

@3nids
Copy link
Member

3nids commented Feb 4, 2022

here is a proposal for a fix #112

@3nids
Copy link
Member

3nids commented Feb 15, 2022

worked as expected
image

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