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

.github/workflows/ci.yml: run on more OS #59

Merged
merged 5 commits into from
Jun 6, 2022
Merged

.github/workflows/ci.yml: run on more OS #59

merged 5 commits into from
Jun 6, 2022

Conversation

flokli
Copy link
Collaborator

@flokli flokli commented Jun 6, 2022

No description provided.

@flokli flokli force-pushed the ci-os branch 7 times, most recently from 2cf4dbd to 86c968a Compare June 6, 2022 17:16
… also add a header_mode_windows.go file.
You can't create fifo files on windows.
@flokli flokli force-pushed the ci-os branch 3 times, most recently from bade049 to 4c5f4e4 Compare June 6, 2022 18:23
While tooling like DumpPath should use `path/filepath` to assemble its paths,
the NAR writer might be called with non-forward slashes both for paths
and symlink targets.
We need to convert these to stay canonical.

The NAR reader returns forward slashes, even when run on windows.
@flokli flokli merged commit b9823f7 into master Jun 6, 2022
@flokli flokli deleted the ci-os branch June 6, 2022 18:55
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.

None yet

2 participants