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

Autotools build improvement #1664

Open
mcuee opened this issue Feb 6, 2024 · 7 comments
Open

Autotools build improvement #1664

mcuee opened this issue Feb 6, 2024 · 7 comments
Labels
enhancement New feature or request

Comments

@mcuee
Copy link
Collaborator

mcuee commented Feb 6, 2024

1) libreadline status
As of now, for auto-tools build, the presence of libreadline is not correct. It always says libreadline is not present even though libreadline is present and the final binary is linked to libreadline.

I tried to fix it last time but it caused issues with macOS terminal mode and I have to revert the following PR.

2) to reduce the warnings:

Reference: many warnings when using autotools build. It is good to fix or at least reduce the warnings.

3) to fix NetBSD Autotools build

Configuration summary:
----------------------
DON'T HAVE libusb
DON'T HAVE libusb_1_0
DON'T HAVE libftdi1
DON'T HAVE libftdi
DON'T HAVE libreadline

4) to fix OpenBSD Autotools build
#1670

Right now there is an error in the last step.

Warning: target ./libavrdude.a (prerequisite of: avrdude) does not have any command (BUG)
  CCLD     avrdude
cc: error: no such file or directory: './libavrdude.a'
*** Error 1 in . (Makefile:1043 'avrdude': @echo "  CCLD    " avrdude;/bin/sh ./libtool --silent --tag=CC    --mode=link cc -Wall -Wextra -W...)
*** Error 1 in . (Makefile:2820 'all-recursive': @fail=;  if (target_option=k; case ${target_option-} in  ?) ;;  *) echo "am__make_running_w...)
*** Error 2 in /home/mcuee/build/avrdude/src (Makefile:886 'all')
@mcuee mcuee added the enhancement New feature or request label Feb 6, 2024
@mcuee mcuee changed the title auto-tools scripts improvement -- update configure.ac to reflect libreadline status Autotools build improvement -- update configure.ac to reflect libreadline status Feb 7, 2024
@mcuee mcuee added the bug Something isn't working label Feb 7, 2024
@stefanrueger
Copy link
Collaborator

@mcuee Thanks for proactively starting issues on things we'd like improved. Can we please use fewer issues: we can summarise what needs improved, eg, have one issue for improving autotools build (listing the 4 aspects to improve) and one issue for improving documentation. It is unlikely we attract a number of people working in parallel on these issues (and for the autotools this is definitely not advisable). Also the new issue on jtag2updi improvement is kinda non-descript. I'd remove that for the time being or make it specific.

Ideally we never grow over a page of issues and discussions so we don't lose sight over things on the second page

@mcuee mcuee changed the title Autotools build improvement -- update configure.ac to reflect libreadline status Autotools build improvement Feb 7, 2024
@mcuee
Copy link
Collaborator Author

mcuee commented Feb 7, 2024

@stefanrueger

Done.

@mcuee
Copy link
Collaborator Author

mcuee commented Feb 7, 2024

@stefanrueger

I have trimmed down the number of issues and discussions to one page. Feel free to close those issues/discussions you think can be closed.

@stefanrueger
Copy link
Collaborator

@mcuee
Brilliant, thank you!

@mcuee
Copy link
Collaborator Author

mcuee commented Feb 11, 2024

The following issue should be present in autotools build as well.

@ndim
Copy link
Contributor

ndim commented Feb 11, 2024

The following issue should be present in autotools build as well.

* [avrintel.h missing from installation #1683](https://github.com/avrdudes/avrdude/issues/1683)

I will reflect in autoconf whatever comes out of the discussion over there.

@mcuee mcuee pinned this issue Mar 17, 2024
@mcuee mcuee unpinned this issue Mar 22, 2024
@mcuee mcuee removed the bug Something isn't working label Apr 26, 2024
@mcuee
Copy link
Collaborator Author

mcuee commented Apr 26, 2024

Remove the bug label now that the remaining portions are for enhancement.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants