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

configure on riscv64 works with updated posix/config.guess and posix/… #18

Closed
wants to merge 1 commit into from

Conversation

ureyNZB
Copy link
Contributor

@ureyNZB ureyNZB commented Sep 14, 2023

…config.sub

Original PR link: nzbget/nzbget#765
Author: @sanderjo

configure on riscv64 works with updated posix/config.guess and posix/config.sub

…config.sub

Original PR link: nzbget/nzbget#765
Author: @sanderjo

configure on riscv64 works with updated posix/config.guess and posix/config.sub
@sanderjo
Copy link
Contributor

sanderjo commented Nov 3, 2023

Instead of accepting my patch, you could decide to do the below now and then in your CD/CI process:

sander@nezha:~/git/nzbget$ 

sudo apt install automake -y
aclocal
automake --add-missing --copy --force-missing

Because the above will update the files like below:

sander@nezha:~/git/nzbget$ head posix/config.guess | grep timestamp
timestamp='2022-01-09'
sander@nezha:~/git/nzbget$ head posix/config.sub | grep timestamp
timestamp='2022-01-03'

... and then everything is fine.

@phnzb
Copy link
Collaborator

phnzb commented Nov 3, 2023

Thank you, good suggestion. We might do that as part of #28

@phnzb
Copy link
Collaborator

phnzb commented Feb 9, 2024

Closing as completed. The configure script has now been removed from the repository and CI jobs use autoreconf.

@phnzb phnzb closed this Feb 9, 2024
@phnzb phnzb deleted the feature-support-posix-riscv64 branch February 9, 2024 14:18
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

3 participants