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

v.db.pyupdate warning during installation #1015

Open
ecodiv opened this issue Jan 13, 2024 · 2 comments
Open

v.db.pyupdate warning during installation #1015

ecodiv opened this issue Jan 13, 2024 · 2 comments
Assignees

Comments

@ecodiv
Copy link
Contributor

ecodiv commented Jan 13, 2024

Name of the addon
v.db.pyupdate

Describe the problem
When installing it through g.extension, I got the following warning:

WARNING: Extension <v.db.pyupdate> already installed. Re-installing...
Fetching <v.db.pyupdate> from <https://github.com/OSGeo/grass-addons/> (be patient)...
Your branch is up to date with 'origin/grass8'.
Already on 'grass8'
Compiling...
Installing...
Updating extensions metadata file...
Updating extension modules metadata file...
Traceback (most recent call last):
  File "/usr/local/grassmaster/grass84/scripts/g.extension",
line 2868, in <module>
    sys.exit(main())
  File "/usr/local/grassmaster/grass84/scripts/g.extension",
line 2848, in main
    install_extension(source=source, url=url, xmlurl=xmlurl,
branch=branch)
  File "/usr/local/grassmaster/grass84/scripts/g.extension",
line 1227, in install_extension
    update_manual_page(module)
  File "/usr/local/grassmaster/grass84/scripts/g.extension",
line 2421, in update_manual_page
    for a in
get_multi_addon_addons_which_install_only_html_man_page():
  File "/usr/local/grassmaster/grass84/scripts/g.extension",
line 1476, in
get_multi_addon_addons_which_install_only_html_man_page
addon_paths.pop(addon_paths.index(addon_src_file.group(0)))
ValueError:
'src/vector/v.db.pyupdate/testsuite/test_v_db_py' is not in
list

The addon installs though, so this is not really a bug. However, it would be good to get rid of the warning.

To Reproduce

Run g.extension extension=v.db.pyupdate operation=add

Expected behavior

Install without the warning messages

System description (please complete the following information):

  • Operating System: Ubuntu 22.04
  • GRASS GIS version 8.4.0dev (64 bit)
  • revision=53c362dbd1
  • build_date=2024-01-05
  • build_platform=x86_64-pc-linux-gnu
  • build_off_t_size=8
  • libgis_revision=53c362dbd1
  • libgis_date=2024-01-05T17:18:52+00:00

Additional context
Add any other context about the problem here.

@wenzeslaus
Copy link
Member

While the "multi addon" part and the involvement of the test_v_db_py directory look like a real issue, I can't reproduce this with current main. I get the Fetching message the first time and Re-installing... message the second time. Can you do more testing? Do other tools behave the same? What is in your ~/.grass8/addons/ directory?

$ grass-dev --tmp-project XY --text
GRASS tmpproject/PERMANENT:grass > g.extension extension=v.db.pyupdate operation=add
Fetching <v.db.pyupdate> from <https://github.com/OSGeo/grass-addons/> (be
patient)...
remote: Enumerating objects: 14, done.
remote: Counting objects: 100% (8/8), done.
remote: Compressing objects: 100% (8/8), done.
remote: Total 14 (delta 0), reused 5 (delta 0), pack-reused 6
Receiving objects: 100% (14/14), 14.26 KiB | 14.26 MiB/s, done.
remote: Enumerating objects: 7, done.
remote: Counting objects: 100% (4/4), done.
remote: Compressing objects: 100% (4/4), done.
remote: Total 7 (delta 0), reused 0 (delta 0), pack-reused 3
Receiving objects: 100% (7/7), 8.41 KiB | 8.41 MiB/s, done.
Already on 'grass8'
Your branch is up to date with 'origin/grass8'.
Compiling...
Installing...
Updating extensions metadata file...
Updating extension modules metadata file...
Installation of <v.db.pyupdate> successfully finished
GRASS tmpproject/PERMANENT:grass > g.extension extension=v.db.pyupdate operation=add
WARNING: Extension <v.db.pyupdate> already installed. Re-installing...
Fetching <v.db.pyupdate> from <https://github.com/OSGeo/grass-addons/> (be
patient)...
remote: Enumerating objects: 14, done.
remote: Counting objects: 100% (8/8), done.
remote: Compressing objects: 100% (8/8), done.
remote: Total 14 (delta 0), reused 5 (delta 0), pack-reused 6
Receiving objects: 100% (14/14), 14.26 KiB | 14.26 MiB/s, done.
remote: Enumerating objects: 7, done.
remote: Counting objects: 100% (4/4), done.
remote: Compressing objects: 100% (4/4), done.
remote: Total 7 (delta 0), reused 0 (delta 0), pack-reused 3
Receiving objects: 100% (7/7), 8.41 KiB | 8.41 MiB/s, done.
Already on 'grass8'
Your branch is up to date with 'origin/grass8'.
Compiling...
Installing...
Updating extensions metadata file...
Updating extension modules metadata file...
Installation of <v.db.pyupdate> successfully finished
GRASS tmpproject/PERMANENT:grass > g.extension extension=v.db.pyupdate operation=remove
Files to be removed:
WARNING: Extension <v.db.pyupdate> not removed. Re-run 'g.extension' with
         '-f' flag to force removal
GRASS tmpproject/PERMANENT:grass > g.extension extension=v.db.pyupdate operation=remove -f
Updating addons metadata file...
Extension <v.db.pyupdate> successfully uninstalled.

@ecodiv
Copy link
Contributor Author

ecodiv commented Dec 14, 2024

@wenzeslaus sorry for the late response.

  • If I remove the ~/.grass8/addons/ directory and install the v.db.pyupdate addon first, I do not get an error message.
  • If I first install another addon and then v.db.pyupdate, I get the error message above. I tried this with various addons (each time deleting the addons directory first, to have clean state).

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