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

Avrdude terminal history broken on macOS #1296

Closed
MCUdude opened this issue Jan 29, 2023 · 5 comments
Closed

Avrdude terminal history broken on macOS #1296

MCUdude opened this issue Jan 29, 2023 · 5 comments
Labels
bug Something isn't working

Comments

@MCUdude
Copy link
Collaborator

MCUdude commented Jan 29, 2023

I just stumbled over this. When building with make, the Avrdude terminal history is broken on macOS.
I've narrowed it down to the following commit by @mcuee: 6595f53

$ ./avrdude -cusbasp -patmega1284p -t

avrdude: AVR device initialized and ready to accept instructions
avrdude: device signature = 0x1e9705 (probably m1284p)
avrdude> read eeprom

Reading | ################################################## | 100% 0.07 s 

0000  48 65 6c 6c 6f 20 57 6f  72 6c 64 00 ff ff ff ff  |Hello World.....|
0010  ff ff ff ff ff ff ff ff  ff ff ff ff ff ff ff ff  |................|
0020  ff ff ff ff ff ff ff ff  ff ff ff ff ff ff ff ff  |................|
0030  ff ff ff ff ff ff ff ff  ff ff ff ff ff ff ff ff  |................|
0040  ff ff ff ff ff ff ff ff  ff ff ff ff ff ff ff ff  |................|
0050  ff ff ff ff ff ff ff ff  ff ff ff ff ff ff ff ff  |................|
0060  ff ff ff ff ff ff ff ff  ff ff ff ff ff ff ff ff  |................|
0070  ff ff ff ff ff ff ff ff  ff ff ff ff ff ff ff ff  |................|
0080  ff ff ff ff ff ff ff ff  ff ff ff ff ff ff ff ff  |................|
0090  ff ff ff ff ff ff ff ff  ff ff ff ff ff ff ff ff  |................|
00a0  ff ff ff ff ff ff ff ff  ff ff ff ff ff ff ff ff  |................|
00b0  ff ff ff ff ff ff ff ff  ff ff ff ff ff ff ff ff  |................|
00c0  ff ff ff ff ff ff ff ff  ff ff ff ff ff ff ff ff  |................|
00d0  ff ff ff ff ff ff ff ff  ff ff ff ff ff ff ff ff  |................|
00e0  ff ff ff ff ff ff ff ff  ff ff ff ff ff ff ff ff  |................|
00f0  ff ff ff ff ff ff ff ff  ff ff ff ff ff ff ff ff  |................|

avrdude> ^[[A^[[A^[[A^[[A^C
@mcuee mcuee added the bug Something isn't working label Jan 29, 2023
@mcuee
Copy link
Collaborator

mcuee commented Jan 29, 2023

Hmm, that is stange. The change is supposed to change nothing but the prompt only. Let me check later.

@mcuee
Copy link
Collaborator

mcuee commented Jan 30, 2023

Strange, git main is broken, even with CMake build. 7.1 release is okay.

Sorry this is a false alarm. I have delete the git repo and try again, it works with CMake.

@mcuee
Copy link
Collaborator

mcuee commented Jan 30, 2023

@MCUdude

Indeed you are right, revert that PR #1280 will indeed fix the issue. I think configure.ac is pretty broken and we may have to really forget about it and only use CMake down the road.

mcuee@mcuees-Mac-mini avrdude_main % git diff
diff --git a/src/configure.ac b/src/configure.ac
index 4f35c1c..6470f88 100644
--- a/src/configure.ac
+++ b/src/configure.ac
@@ -83,7 +83,7 @@ AC_CHECK_LIB([ws2_32], [WSAStartup])
 
 AC_CHECK_LIB([termcap], [tputs])
 AC_CHECK_LIB([ncurses], [tputs])
-AC_CHECK_LIB([readline], [readline], [have_readline=yes])
+AC_CHECK_LIB([readline], [readline])
 AH_TEMPLATE([HAVE_LIBELF],
             [Define if ELF support is enabled via libelf])
 AC_CHECK_LIB([elf], [elf_getshdrstrndx], [have_libelf=yes])

mcuee added a commit to mcuee/avrdude that referenced this issue Jan 30, 2023
@mcuee
Copy link
Collaborator

mcuee commented Jan 30, 2023

@MCUdude

I have created PR #1297 to revert the change of PR #1280.

@mcuee
Copy link
Collaborator

mcuee commented Jan 31, 2023

@MCUdude

I have merged #1297 to fix this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants