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

159 fix spec builds local and gh action #160

Closed
wants to merge 4 commits into from

Conversation

wmat
Copy link

@wmat wmat commented Aug 8, 2024

Updated build to use Docker both for local builds and the GH Action build.

Copy the yml from the docs-template to build using Docker.
Rename the file as build-pdf.yml.
Fix makefile and header file to build.
Add a wildcard to run action on any branch push.
Removed table font from header and updated version in Makefile.
@wmat wmat requested a review from atishp04 August 8, 2024 18:47
@wmat wmat linked an issue Aug 8, 2024 that may be closed by this pull request
@avpatel
Copy link
Contributor

avpatel commented Aug 9, 2024

This PR simply throws away all the Makefile improvements that we had done in the past which is not acceptable.

@wmat
Copy link
Author

wmat commented Aug 9, 2024

This is how most of the other specs in the risc-v family of documents are setup. However, if you'd prefer to stick with your Makefile setup we'll have to find another way to fix the build.

@avpatel
Copy link
Contributor

avpatel commented Aug 15, 2024

@wmat I have another PR (#162) to fix this issue which tries to keep the functionality in Makefile as much as possible.

@wmat
Copy link
Author

wmat commented Aug 15, 2024

Cool. Feel free to kill off my PR.

@atishp04
Copy link
Collaborator

Closing this PR as the other PR #162 is already merged.

@atishp04 atishp04 closed this Aug 30, 2024
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.

Fix spec builds, local and GH action
3 participants