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

New package: radm-0.6.4 #38839

Closed
wants to merge 1 commit into from
Closed

New package: radm-0.6.4 #38839

wants to merge 1 commit into from

Conversation

Goorzhel
Copy link
Contributor

  • I tested the changes in this PR: YES
  • This new package (however self-serving) conforms to the package requirements: YES

@classabbyamp classabbyamp added the new-package This PR adds a new package label Aug 22, 2022
@Goorzhel
Copy link
Contributor Author

Goorzhel commented Aug 22, 2022

Both CI failures had to do with the chvt crate. I'll try to reproduce this locally (

❯ for arch in {x86_64,armv{6,7}l,i686,aarch64}{,-musl}; do
  ./xbps-src pkg -a $arch $(git b) &> $arch.log
done

) and submit any necessary patches to the crate's upstream.

@Goorzhel
Copy link
Contributor Author

Also, a majority of the architectures on this run returned success, but actually failed with

=> ERROR: pandoc-2.17.1.1_1: cannot be cross compiled...
=> ERROR: pandoc-2.17.1.1_1: yes

because I mistakenly added pandoc to makedepends rather than hostmakedepends. I should investigate that later, too.

@Goorzhel
Copy link
Contributor Author

Patches sent upstream to unbreak non-Intel builds:
jdonszelmann/chvt-rs#2
1wilkens/pam#36

I still need to test these changes at runtime so I'll leave the branch unsquashed for now.

@Goorzhel Goorzhel changed the title New package: radm-0.6.1 New package: radm-0.6.3 Aug 23, 2022
@Goorzhel
Copy link
Contributor Author

Goorzhel commented Aug 23, 2022

Also, a majority of the architectures on this run returned success, but actually failed

Turns out build.sh was eating the exit code. #38841 I'm an idiot.

@q66
Copy link
Contributor

q66 commented Aug 23, 2022

pandoc is not portable, put the manpage behind a build option that is only enabled if the host architecture is one that has it (x86_64*|i686|ppc64le*|ppc64)

@Goorzhel
Copy link
Contributor Author

This is as done as it's ever going to be. Squashing.

@Goorzhel Goorzhel changed the title New package: radm-0.6.3 New package: radm-0.6.4 Sep 1, 2022
@Goorzhel Goorzhel closed this Nov 13, 2022
@Goorzhel Goorzhel deleted the radm branch November 13, 2022 22:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new-package This PR adds a new package
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants