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

La fonction next_git_tag() nuie au travail sur le code #374

Closed
MaelREBOUX opened this issue Aug 1, 2022 · 2 comments · Fixed by #375
Closed

La fonction next_git_tag() nuie au travail sur le code #374

MaelREBOUX opened this issue Aug 1, 2022 · 2 comments · Fixed by #375
Assignees
Labels
bug Python remonte une erreur pur Python

Comments

@MaelREBOUX
Copy link
Collaborator

MaelREBOUX commented Aug 1, 2022

Description du bug

Bonjour @Gustry,

Nous travaillons sur #370 et #372 et nous avons des difficultés à tester les modifications.
En effet le commit 5b8edfd introduit des fonctions qui font appel à git describe.... Or si on prend / copie / alias le répertoire cadastre du projet pour qu'il soit visible : cela produit immanquablement l'erreur suivante :

Traceback (most recent call last):
  File "C:/Users/m.reboux/AppData/Roaming/QGIS/QGIS3\profiles\default/python/plugins\cadastre\cadastre_menu.py", line 259, in open_import_dialog
    dialog = CadastreImportDialog(self.iface)
  File "C:/Users/m.reboux/AppData/Roaming/QGIS/QGIS3\profiles\default/python/plugins\cadastre\dialogs\import_dialog.py", line 36, in __init__
    self.setWindowTitle('{} {}'.format(self.windowTitle(), set_window_title()))
  File "C:/Users/m.reboux/AppData/Roaming/QGIS/QGIS3\profiles\default/python/plugins\cadastre\tools.py", line 90, in set_window_title
    return 'next {}'.format(next_git_tag())
  File "C:/Users/m.reboux/AppData/Roaming/QGIS/QGIS3\profiles\default/python/plugins\cadastre\tools.py", line 73, in next_git_tag
    tag = git_show.communicate()[0].partition('\n')[0]
  File "C:\PROGRA~1\QGIS\3.16\apps\Python37\lib\subprocess.py", line 920, in communicate
    stdout, stderr = self._communicate(input, endtime, timeout)
  File "C:\PROGRA~1\QGIS\3.16\apps\Python37\lib\subprocess.py", line 1267, in _communicate
    stderr = stderr[0]
IndexError: list index out of range

Proposition de résolution

Je suppose que les développeurs "pros" (je ne m'inclue pas dedans) travaillent dans des environnements de développement correctement configurés.
Pour autant je ne comprend pas comment la version actuelle du plugin (v 1.14.2) téléchargée depuis le gestionnaire de plugin fonctionne sachant qu'il n'y a pas le répertoire supérieur et que ce n'est PAS un repo git fonctionnel ??

Je vais donc proposer une gestion de l'erreur si on tente un git describe....

Environnement

  • OS: Windows 10
  • Version de QGIS : 3.10 / 3.16
  • Version du plugin : code source issu du clone de dépôt
@MaelREBOUX MaelREBOUX added the bug label Aug 1, 2022
@MaelREBOUX MaelREBOUX self-assigned this Aug 1, 2022
@MaelREBOUX MaelREBOUX added the Python remonte une erreur pur Python label Aug 1, 2022
@MaelREBOUX
Copy link
Collaborator Author

Testé et validé. Ok pour merge selon moi.

@MaelREBOUX
Copy link
Collaborator Author

merci @Gustry

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Python remonte une erreur pur Python
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant