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

Regenerate html pages for #3052 #3054

Merged
merged 3 commits into from
Mar 1, 2024
Merged

Conversation

Jakuje
Copy link
Member

@Jakuje Jakuje commented Feb 29, 2024

The HTML pages contain also some paths that were sometimes pointing in wrong direction. I used the standard prefix "/usr"` for the configure to get the sane paths:

./configure --prefix="/usr"
make -j8
cd doc/tools
make tools.html
cd ../files
make files.html

I also removed trailing whitespace from source documentation xmls before generating the HTML.

Note, that the HTML diff might look weird when there are non-ascii chararacters in the UTF8 terminals, but the HTML page has ISO-8859-1 defined in the HTML Header so it should render ok.

Checklist
  • Documentation is added or updated

@Jakuje
Copy link
Member Author

Jakuje commented Feb 29, 2024

The original files had forced default ISO encoding, which did not play well with the javascript page written in utf8 forcing this encoding. In 21st century everything should be able to handle UTF8. The changes needed to do this are described in

http:https://www.sagehill.net/docbookxsl/OutputEncoding.html

Applying them to the build process generates valid UTF8 files:

http:https://htmlpreview.github.io/?https://github.com/OpenSC/OpenSC/blob/9c8aeaabc964f8875ad272686db09f1f073454c6/doc/tools/tools.html

@Jakuje Jakuje mentioned this pull request Feb 29, 2024
5 tasks
@Jakuje Jakuje added this to In progress in OpenSC 0.25.0 via automation Feb 29, 2024
@frankmorgner frankmorgner merged commit 5a9b18a into OpenSC:master Mar 1, 2024
36 of 37 checks passed
OpenSC 0.25.0 automation moved this from In progress to Done Mar 1, 2024
@frankmorgner
Copy link
Member

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

2 participants