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

Fix example + distcheck #120

Merged
merged 3 commits into from
Nov 10, 2023
Merged

Fix example + distcheck #120

merged 3 commits into from
Nov 10, 2023

Conversation

vasi
Copy link
Owner

@vasi vasi commented Nov 9, 2023

  • Add extra dependencies to README example, it should work now in a docker ubuntu:jammy container
  • While testing that this was fixed, noticed that out-of-tree builds and tests from release tarballs are both broken!
    • Added some missing files to the dist tarball
    • Made ll-smoke-singlethreaded aware of where ll-smoke lives, so it can be run out of tree
    • Added a new test to enforce that dist builds are working

@vasi vasi force-pushed the update-example2 branch 2 times, most recently from c6c61e8 to bfc5ab0 Compare November 9, 2023 17:49
* Include some missing test files in dist (+ some docs)
* Make ll-smoke-singlethreaded not get confused by out-of-tree builds
* Run distcheck in CI
@vasi vasi changed the title WIP: Fix example + distcheck Fix example + distcheck Nov 9, 2023
@vasi vasi mentioned this pull request Nov 9, 2023
@vasi
Copy link
Owner Author

vasi commented Nov 9, 2023

@DrDaveD Does this look reasonable?

Copy link
Collaborator

@DrDaveD DrDaveD left a comment

Choose a reason for hiding this comment

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

Looks good to me

@vasi vasi merged commit 2092408 into master Nov 10, 2023
7 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.

2 participants