Improve autotools build (part 2/n) #1695
Draft
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Yes CMake does build ps file. Example run from git main (Linux x86_64 github action build). It also installs ps file. |
ndim
force-pushed
the
improve-autotools-build-part-2
branch
2 times, most recently
from
eb8130a
to
849fa5e
Compare
|
Strange, somehow the auto-tools build is also broken on my FreeBSD 14 Release system (virtual machine under Proxmox PVE 8.0, Intel N100 mini PC). Two issues.
And it is the same for git main or avrdude-7.3 release. I remember it was working last time, probably on another FreeBSD system. Please check on your FreeBSD system as well. If it is broken, then please add the task to fix FreeBSD autotools build as well. BTW, I am using latest git main (5c61b9e) and not this PR. build log under FreeBSD 14 for git mainNo issues to use CMake. CMake build log of git main |
|
One thing to try on systems where GNU make is not the default Also, the non-libtool library needs to go. |
|
Ah, I see. Once I use gmake, the build is fine. Now the only issue is why libserialport is not detected. using gmake and the build is fine |
|
Not using pkgconfig for anything yet. |
I understand. On the other hand, I do not see why libserialport is special in this aspect, it is the same for other dependancies. BTW, it is a bit strange that I need to use |
gmake was basically already required before, in particular when trying to run parallel jobs ( |
This transfers the information from the CMakeLists.txt libavrdude VERSION and SOVERSION into the autotools configure script and AC_SUBST into every Makefile. This information is completely unused at this time.
* move autotools package related files to top_srcdir:
configure.ac, avrdude.spec.in, bootstrap
* ship all files required to build with cmake
* new tar format for dist tarball allowing longer filenames
for the tools/test_files/*.hex files
* new dist tarball compression method to make it use a different
filename `*.tar.xz` than github's snapshot and release
tarballs `*.tar.gz`
ndim
force-pushed
the
improve-autotools-build-part-2
branch
from
849fa5e
to
77edcdb
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This is a continuation which is based on #1681 to keep the PR sizes manageable both for the submitter and the reviewers.
Starting this as a Draft PR, expect rebases and force-pushing, e.g. rebasing to main after PR #1681 has been merged to main.
Some goals for this second round of fixing the autotools build:
avrdude.ps?)--enable-versioned-doc, have people use--docdirto deviate from the default docdir*.texifiles-Walloption_LIBRARIES)Concerning both cmake and autotools builds:
Optionally, move
$top_srcdir/src/configure.acto$top_srcdir/configure.ac:README.md,NEWS, and especiallyCOPYINGintomake disttarballmake checkruntools/test-avrdude, somake distcheckwould cover everything in CI builds and the compile and install stages will only need to run once duringmake distcheck(about halves the CPU cycles needed for autotools CI builds).tar.xzand the Github tarballs.tar.gz