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

Issue 1059: Fix downloadable abs path #1060

Merged
merged 2 commits into from
Nov 9, 2024

Conversation

Paldari
Copy link
Contributor

@Paldari Paldari commented Nov 6, 2024

fixes #1059

This is quick fix for the regression discussed in issue 1059.

@Paldari Paldari force-pushed the bugfix/download-absolute-path branch from 5b8bdc3 to afe1ab6 Compare November 6, 2024 04:55
@Paldari
Copy link
Contributor Author

Paldari commented Nov 6, 2024

Ha... the test I added is obviously failing on windows since the path separator I used in the test is the Unix one... I think even on Windows a lot of people would actually use the Unix separator, so I should update the PR.

@Paldari Paldari force-pushed the bugfix/download-absolute-path branch 3 times, most recently from 134f815 to 73e28ff Compare November 8, 2024 00:20
@Paldari Paldari marked this pull request as draft November 8, 2024 03:24
@Paldari Paldari force-pushed the bugfix/download-absolute-path branch from 73e28ff to de10ee0 Compare November 8, 2024 04:10
@Paldari Paldari marked this pull request as ready for review November 8, 2024 05:59
@Paldari
Copy link
Contributor Author

Paldari commented Nov 8, 2024

PR should be fine for review now.

Issue was just that a path starting with / or \ is not an absolute path on windows (it must start with a drive). Kind of obvious but still took me several tries to understand without a local Windows env, haha
I could add a comment about that if you deem it necessary.

Cheers

A series of download entries have been added to help verify
project-relative paths are resolved. Extended the Confluence-related
checks to verify that expected view-file macros are generated.

Signed-off-by: James Knight <[email protected]>
@jdknight jdknight merged commit 9655599 into sphinx-contrib:main Nov 9, 2024
24 checks passed
@jdknight
Copy link
Member

jdknight commented Nov 9, 2024

Thanks for taking the time to address the issue and present a clean/signed change request, as well as reporting the issue. Sorry for the regression on this. Hopefully future changes won't break this again. Merged; and it should be available next release.

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.5+] Downloadable files not found anymore when using sphinx 'absolute' paths (in truth relative to 'src')
2 participants