-
Notifications
You must be signed in to change notification settings - Fork 415
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
lvfs-testing: no signature method in results #2223
Comments
Did you compile libjcat without GPG and GNUTLS? What libjcat version are you using? |
|
libjcat 0.1.3 It is built with both gpg an gnutls |
|
|
This might be a clue: |
I think you're using a fwupd built against the old version with a new version of jcat. |
Yes, that was it :) |
|
fwuptool still complains though(fwupdmgr refresh is OK): |
It's supposed to be. I had a thinko in the last release. I think we can work around it in fwupd. |
Some deatils: |
|
Regarding the assertion, can you check what |
|
If I was going to guess what's happening is that your |
Right, it wasn't set: |
|
Would you be able to walk through this with a debugger and see if my hypothesis looks correct in your scenario? |
Will this do? |
|
Some more data, I think this is causing it: |
|
|
if move away vendor.conf and vendor-directory.conf, the error goes away |
|
This is annoying though: |
|
This seems to do the trick: |
On Gentoo, UEFI pulls in tpm2-tss. Is that really needed? Gentoo has a special USE=tpm which also pulls in tpm2-tss |
|
Thanks for digging into it. I think that approach works, but we should just be checking the remote type and only refreshing download type remotes. |
|
Hmm, we do have a check for |
|
And also another check that the remote is enabled too.. |
Sure, but ... |
|
That's true. I think we should do your fix, would you mind doing a proper PR for it? |
Needs to be |
If I could get away without, it will take anothe 30 mins to get setup to do that. Please?
|
Why? other cases just return FALSE in that same function and uses !xxx |
|
|
FYI, the scanning stop after the first error so vendor-directory.conf is never check |
@joakim-tjernlund can you do a pretty complex thing for me please?
Thanks. If that works I'll commit the fix to libjcat to restore the old ABI. |
Did that and fwupdtool refresh worked fine in step 4. :) |
Much appreciated. If Mario approves of the workaround I'll commit to libjcat master. |
|
@hughsie can you backport the fwupd one to 1_4_X for @joakim-tjernlund? |
I am waiting for Lenovo Dock support so I will go to 1.5 as soon it is out(which I hope is soon). |
We're aiming on getting all the docks supported in 1.4.x too. Most of it is there already. |
Describe the bug
fwupdmgr refresh
Fetching signature https://cdn.fwupd.org/downloads/firmware-testing.xml.gz.jcat
Downloading… []
Fetching metadata https://cdn.fwupd.org/downloads/firmware-00432-testing.xml.gz
Downloading… []
Failed to update metadata for lvfs-testing: no signature method in results
fwupd version information
fwupdmgr --version
client version: 1.4.4
compile-time dependency versions
gusb: 0.3.4
efivar: 37
daemon version: 1.4.4
emerge sys-apps/fwupd
Additional questions
The text was updated successfully, but these errors were encountered: