SSIA ^_^
Created attachment 434515 [details] Patch
Created attachment 434516 [details] Patch v2
Committed r280423 (240063@main): <https://commits.webkit.org/240063@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 434516 [details].
Reopening for debug build fixes regarding gpg-error not being pulled as dependency automatically by libgcrypt.
(In reply to Lauro Moura from comment #4) > Reopening for debug build fixes regarding gpg-error not being pulled as > dependency automatically by libgcrypt. Gah! I assumed that libgcrypt.pc would list gpg-error as a dependency, but it does not — which is what manually adding -lgpg-error was workarounding. I will submit a patch adding also a LibGcrypt::GpgError imported target and make LibGcrypt::LibGcrypt depend on it :|
Created attachment 434559 [details] Follow-up Patch
Build failure here: https://build.webkit.org/#/builders/43/builds/3684
Committed r280461 (240096@main): <https://commits.webkit.org/240096@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 434559 [details].
I'm afraid we would have to revert this, or at least modify it to not rely on the pkg-config files After r280423 is not longer possible to build WebKit on Debian 10. See the failure on our bot: https://build.webkit.org/#/builders/46/builds/5192 I can reproduce this on buildbox with: $ rm -fr WebKitBuild # wipe build so there is no flatpak $ Tools/Scripts/build-webkit --gtk --no-experimental-features It seems the issue is that the package that Debian 10 ships for libgcrypt20-dev doesn't have a libgcrypt.pc file, and I can't find this file shipped by any other package on the distro. I can see on the libgcrypt upstream repository that the pc file was added in $ git describe 97194b422bc89a6137f4e218d4cdee118c63e96e libgcrypt-1.9-base-91-g97194b42 And Debian 10 still ships libgcrypt 1.8.4
(In reply to Carlos Alberto Lopez Perez from comment #9) > I'm afraid we would have to revert this, or at least modify it to not rely > on the pkg-config files > > > After r280423 is not longer possible to build WebKit on Debian 10. > > See the failure on our bot: > https://build.webkit.org/#/builders/46/builds/5192 > > I can reproduce this on buildbox with: > > $ rm -fr WebKitBuild # wipe build so there is no flatpak > $ Tools/Scripts/build-webkit --gtk --no-experimental-features > > It seems the issue is that the package that Debian 10 ships for > libgcrypt20-dev doesn't have a libgcrypt.pc file, and I can't find this file > shipped by any other package on the distro. > > I can see on the libgcrypt upstream repository that the pc file was added in > > $ git describe 97194b422bc89a6137f4e218d4cdee118c63e96e > libgcrypt-1.9-base-91-g97194b42 > > And Debian 10 still ships libgcrypt 1.8.4 Not having a .pc file should not be a failure, there is a fallback path that uses the “libgcrypt-config” script, which is part of the Debian package in versions <1.9, see: https://packages.debian.org/buster/amd64/libgcrypt20-dev/filelist From the build log you linked I see that the fallback is being tried but I made some mistake there. I would not revert; so let's reopen this and I will submit a follow-up with a fix in the next hour or so :)
Created attachment 434608 [details] Follow-up Patch (bis)
Comment on attachment 434608 [details] Follow-up Patch (bis) nice! thanks!
Committed r280469 (240104@main): <https://commits.webkit.org/240104@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 434608 [details].