Skip to content

Fix syntax error architecture#28

Merged
pbuncic merged 2 commits into
mainfrom
fix-syntax-error-architecture
Oct 17, 2025
Merged

Fix syntax error architecture#28
pbuncic merged 2 commits into
mainfrom
fix-syntax-error-architecture

Conversation

@Ni-Langguth

Copy link
Copy Markdown

Previously on alma10 (ignore the alma9-physical machine, this is in a docker):

[root@lcgapp-alma9-physical lcg.bits]# bits architecture
/build/nlanggut/bitsPlayground/bits/bits_helpers/sync.py:344: SyntaxWarning: invalid escape sequence '\;'
  err = execute("""\
slc10_x86-64

Now on alma10:

[root@lcgapp-alma9-physical bits]# bits architecture
slc10_x86-64

Also err was not used anywhere, as shown here:

[root@lcgapp-alma9-physical bits]# ruff check bits_helpers/sync.py
F841 Local variable `err` is assigned to but never used
   --> bits_helpers/sync.py:344:5
    |
343 |     cvmfs_architecture = re.sub(r"slc(\d+)_x86-64", r"el\1-x86_64", self.architecture)
344 |     err = execute("""\
    |     ^^^
345 |     set -x
346 |     # Exit without error in case we do not have any package published
    |
help: Remove assignment to unused variable `err`

So I added a print at the end of the function.

@Ni-Langguth Ni-Langguth self-assigned this Oct 17, 2025
@Ni-Langguth
Ni-Langguth requested a review from pbuncic October 17, 2025 12:40
@pbuncic
pbuncic merged commit 13b1fd1 into main Oct 17, 2025
0 of 9 checks passed
@andresailer
andresailer deleted the fix-syntax-error-architecture branch October 17, 2025 13:56
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

Successfully merging this pull request may close these issues.

2 participants