Skip to content

utils: Don't set executable bit on mkdocs.py.#7325

Merged
nilason merged 1 commit intoOSGeo:mainfrom
sebastic:mkdocs-install
Apr 17, 2026
Merged

utils: Don't set executable bit on mkdocs.py.#7325
nilason merged 1 commit intoOSGeo:mainfrom
sebastic:mkdocs-install

Conversation

@sebastic
Copy link
Copy Markdown
Contributor

It's not an executable script.

@echoix
Copy link
Copy Markdown
Member

echoix commented Apr 17, 2026

Is this related to the 8.5 RC?

@nilason nilason changed the title Don't set executable bit on mkdocs.py. utils: Don't set executable bit on mkdocs.py. Apr 17, 2026
@nilason nilason added this to the 8.5.0 milestone Apr 17, 2026
@echoix
Copy link
Copy Markdown
Member

echoix commented Apr 17, 2026

Is the executable bit correctly not stored in the git repo (it’s the only Linux attribute that git stores). I think it’s not set, but I’m not sure if I’m looking at the right place in the GitHub UI.

@nilason
Copy link
Copy Markdown
Contributor

nilason commented Apr 17, 2026

Thanks! This will go to RC2 or final release, RC1 is already in the pipeline.

@echoix
Copy link
Copy Markdown
Member

echoix commented Apr 17, 2026

Thanks! This will go to RC2 or final release, RC1 is already in the pipeline.

He probably got notified by the tag created, and the packaging tools spotted that

@sebastic
Copy link
Copy Markdown
Contributor Author

Is this related to the 8.5 RC?

Yes, the Lintian QA tool reported an executable-not-elf-or-script issue for the Debian package build of 8.5.0RC1.

Is the executable bit correctly not stored in the git repo (it’s the only Linux attribute that git stores). I think it’s not set, but I’m not sure if I’m looking at the right place in the GitHub UI.

The file in the tarball is fine, the install commands adds the executable bit:

$ tar tavf ../grass-8.5.0RC1.tar.gz | grep utils/mkdocs.py
-rw-r--r-- runner/runner   14799 2026-04-17 13:11 grass-8.5.0RC1/utils/mkdocs.py

@nilason nilason added the backport to 8.5 PR needs to be backported to release branch 8.5 label Apr 17, 2026
@nilason nilason enabled auto-merge (squash) April 17, 2026 19:17
@nilason nilason merged commit 300baac into OSGeo:main Apr 17, 2026
27 of 28 checks passed
@sebastic sebastic deleted the mkdocs-install branch April 17, 2026 19:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport to 8.5 PR needs to be backported to release branch 8.5

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants