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

Bumb readthedocs ubuntu image version to fix the docs generation #1935

Merged
merged 1 commit into from
Dec 19, 2024

Conversation

kontura
Copy link
Contributor

@kontura kontura commented Dec 6, 2024

The old ubuntu-22.04 uses version swig4.0 which fails on C++11 attributes. We have started to use them in: #1793 and the readthedocs build has been failing since then.

Use 24.04 (https://about.readthedocs.com/blog/2024/06/ubuntu-24-04/) because it uses swig_4.2.0 which at least ignores the attributes.

According to the changelog the attributes should be ignored since version 4.1.0: https://www.swig.org/Release/CHANGES

2022-07-07: jmarrec
	    #1158 #2286 Add basic support for C++11 attributes.  These are now
	    crudely ignored by SWIG's parser's tokeniser, which is better than
	    failing with a parse error.

The old `ubuntu-22.04` uses version `swig4.0` which fails on C++11 attributes.
We have started to use them in: rpm-software-management#1793
and the readthedocs build has been failing since then.

Use `24.04` (https://about.readthedocs.com/blog/2024/06/ubuntu-24-04/)
because it uses `swig_4.2.0` which at least ignores the attributes.

According to the changelog the attributes should be ignored since
version 4.1.0: https://www.swig.org/Release/CHANGES
```
2022-07-07: jmarrec
	    rpm-software-management#1158 #2286 Add basic support for C++11 attributes.  These are now
	    crudely ignored by SWIG's parser's tokeniser, which is better than
	    failing with a parse error.
```
@kontura
Copy link
Contributor Author

kontura commented Dec 6, 2024

The failing tests are due to: rpm-software-management/ci-dnf-stack#1601

@ppisar
Copy link
Contributor

ppisar commented Dec 6, 2024

The failing tests are due to: rpm-software-management/ci-dnf-stack#1601

Shouldn't we first resolve that?

Copy link
Member

@jan-kolarik jan-kolarik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@jan-kolarik jan-kolarik added this pull request to the merge queue Dec 19, 2024
Merged via the queue into rpm-software-management:main with commit d51f614 Dec 19, 2024
17 checks passed
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.

3 participants