-
Notifications
You must be signed in to change notification settings - Fork 42
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
tests: explicitly require libsoup 2.x #326
base: master
Are you sure you want to change the base?
tests: explicitly require libsoup 2.x #326
Conversation
Good catch! One thing though, setting the version explicitly to 2.4 fails the macos tests. Can you find which is the package version for macos and set it for that? It might be necessary to use |
According to https://formulae.brew.sh/formula/libsoup and node-gtk/.github/workflows/install.sh Line 15 in 46a5aac
TypeError: Soup.URI is not a constructor . I think CI for MacOS should also be using v2 and tests can be updated for v3 compatibility.
|
As current tests are not compatible with v3. Signed-off-by: Mateusz Gozdek <mgozdekof@gmail.com>
As if libsoup3 is installed together with libsoup2, libsoup3 is selected which makes test fails. Signed-off-by: Mateusz Gozdek <mgozdekof@gmail.com>
6585ea3
to
b8734ce
Compare
Hmm, on my Arch libsoup 2.74.2 installs libsoup 2.4: $ pacman -Ql libsoup | grep 2.4.so
libsoup /usr/lib/libsoup-2.4.so
libsoup /usr/lib/libsoup-2.4.so.1
libsoup /usr/lib/libsoup-2.4.so.1.11.2
libsoup /usr/lib/libsoup-gnome-2.4.so
libsoup /usr/lib/libsoup-gnome-2.4.so.1
libsoup /usr/lib/libsoup-gnome-2.4.so.1.11.2 I'll check on MacOS when I have some time.. |
Same here ... I have this
but then the installed one is named 2.4 ... and that's expected though |
Same on Ubuntu: https://packages.ubuntu.com/jammy/amd64/libsoup2.4-1/filelist. But MacOS formula also mentions 2.4: https://github.com/Bo98/homebrew-core/blob/master/Formula/libsoup@2.rb |
Also, perhaps those should be unified: $ git grep "gi.require('Soup'"
tests/conversion__g_hash.js:const soup = gi.require('Soup')
tests/conversion__g_list.js:const Soup = gi.require('Soup')
tests/conversion__g_s_list.js:const soup = gi.require('Soup')
tests/function_call__invalid_type.js:const Soup = gi.require('Soup')
tests/object__initialization.js:const Soup = gi.require('Soup', '2.4')
tests/regressions.js:const soup = gi.require('Soup', '2.4') Or there should be separate set of test for 3.x. I can also reproduce the failing tests on MacOS. |
As if libsoup3 is installed together with libsoup2, libsoup3 is selected
which makes test fails.
Signed-off-by: Mateusz Gozdek mgozdekof@gmail.com