Skip to content

DO NOT MERGE: Trying to debug TestContext_Verify#48

Closed
mtrmac wants to merge 2 commits into
proglottis:masterfrom
mtrmac:debug
Closed

DO NOT MERGE: Trying to debug TestContext_Verify#48
mtrmac wants to merge 2 commits into
proglottis:masterfrom
mtrmac:debug

Conversation

@mtrmac
Copy link
Copy Markdown
Contributor

@mtrmac mtrmac commented Nov 5, 2025

My recent pull requests show macOS failures in that function, while it seems to run reliably for me locally.

@mtrmac mtrmac force-pushed the debug branch 4 times, most recently from eba207c to 763c478 Compare November 6, 2025 19:10
Signed-off-by: Miloslav Trmač <mitr@redhat.com>
@mtrmac
Copy link
Copy Markdown
Contributor Author

mtrmac commented Nov 6, 2025

OK the

ld: warning: '/private/var/folders/xc/…/T/go-link-…/go.o' has malformed LC_DYSYMTAB, expected 73 undefined symbols to start at index 4998, found 133 undefined symbols starting at index 37

warning goes away after using a recent Go version.

That doesn’t fix the TestContext_Verify test.

Signed-off-by: Miloslav Trmač <mitr@redhat.com>
@mtrmac
Copy link
Copy Markdown
Contributor Author

mtrmac commented Nov 6, 2025

@proglottis comparing the test results from 601ef89 and da702f4 , updating GnuPG fixes TestContext_Verify; the previous version seems to just not output the data and quit, for some reason.

Not finding the root cause is unsatisfactory, but I’m also not very motivated to diagnose bugs in 7-year-old unmaintained software.

mtrmac added a commit to mtrmac/gpgme that referenced this pull request Nov 6, 2025
The last release was 1.4.23 in Jun 2018; it's not very
relevant in modern environments.

More importantly, the unit tests have been consistently failing
on macOS recently, and per proglottis#48
it seems that the underlying GnuPG does not provide the expected output,
and updating GnuPG fixes that.

We've been pinned to 1.x for ctx.SetCallback, but that feature
now exists in 2.x as well, as long as callers opt in. So, do so.

Signed-off-by: Miloslav Trmač <mitr@redhat.com>
@mtrmac mtrmac closed this Nov 6, 2025
@mtrmac mtrmac mentioned this pull request Nov 6, 2025
@mtrmac
Copy link
Copy Markdown
Contributor Author

mtrmac commented Nov 6, 2025

The log for 601ef89 contains two instances of _gpgme_io_close:305: error: Bad file descriptor (9), which worries me.

That’s also gone with the newer GnuPG.

@mtrmac
Copy link
Copy Markdown
Contributor Author

mtrmac commented Nov 6, 2025

For the record, I can reproduce both the test failure, and the Bad file descriptor error, locally, when installing gnupg@1.4 on macOS.

@mtrmac
Copy link
Copy Markdown
Contributor Author

mtrmac commented Nov 6, 2025

For the record, I can reproduce both the test failure, and the Bad file descriptor error, locally, when installing gnupg@1.4 on macOS.

… and, logging the status of gpgme’s notify_table, I’m fairly convinced that that’s a bug somewhere in that codebase. I will pursue investigating / fixing that through other channels.

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.

1 participant