Skip to content

netsurf: 3.5 -> 3.8 #54610

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

Closed
wants to merge 13 commits into from
Closed

netsurf: 3.5 -> 3.8 #54610

wants to merge 13 commits into from

Conversation

samueldr
Copy link
Member

Motivation for this change

Updates netsurf and related libraries.

The default config for netsurf (using sixel ui) wasn't tested, I can't get it to work, in the previous iteration either. The gtk ui, though works fine.

Things done
  • ✔️ Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
  • Built on platform(s)
    • ✔️ NixOS
    • 🔲 macOS
    • 🔲 other Linux distributions
  • 🔲 Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • ✔️ Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
  • ✔️ Tested execution of all binary files (usually in ./result/bin/)
  • 🔲 Determined the impact on package closure size (by running nix path-info -S before and after)
  • ✔️ Assured whether relevant documentation is up to date
  • ✔️ Fits CONTRIBUTING.md.

@GrahamcOfBorg GrahamcOfBorg requested a review from vrthra January 26, 2019 03:21
@GrahamcOfBorg GrahamcOfBorg added 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 11-100 This PR causes between 11 and 100 packages to rebuild on Linux. labels Jan 26, 2019
@Mic92
Copy link
Member

Mic92 commented Jan 26, 2019

@GrahamcOfBorg build netsurf

@infinisil infinisil added the 8.has: package (update) This PR updates a package to a newer version label Mar 30, 2019
@mmahut
Copy link
Member

mmahut commented May 19, 2019

@GrahamcOfBorg build netsurf

@fgaz
Copy link
Member

fgaz commented Jul 2, 2019

Ping :)

What is the status of this? Does it need more testing?

@samueldr
Copy link
Member Author

samueldr commented Jul 3, 2019

It has some issues, but fewer than the previous version:

Tried to load up GitHub:

image

I had some (4) with nixos.org, but see, it works...

image

This is an improvement to the current behaviour of not working after showing empty windows.

image

So... erm I'm not sure it was acting that way back in January. Not sure where to go from here.

@S-NA
Copy link
Contributor

S-NA commented Jul 3, 2019

According to the Netsurf mailing list, the last call for fixes to land in 3.9 is happening now. The version is expected to fix a lot of issues. Including incompatibilities with newer versions of cURL which I encounter when running Netsurf using this PR.

I have an environment with nix-shell for developing Netsurf and the build it produces seems to work with no issues. Maybe merge this and make a note to update to the 3.9 release soon or perhaps wait, since the release is not too far off. That said feedback is requested against their CI builds.

@fgaz
Copy link
Member

fgaz commented Jul 3, 2019

+1 for merging this for now and seeing what happens with 3.9 once it's released

@samueldr
Copy link
Member Author

samueldr commented Jul 3, 2019

Let's wait; no change here is worth it since it's likely close; 3.9 is likely to fix the issue if I understand it right.

@samueldr
Copy link
Member Author

Turns out the update didn't fix the issue with the certificates being unknown, but I figured out a solution to the problem. See #65360.

@samueldr samueldr closed this Jul 25, 2019
@samueldr samueldr deleted the updates/netsurf-3.8 branch November 4, 2024 16:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
8.has: package (update) This PR updates a package to a newer version 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 11-100 This PR causes between 11 and 100 packages to rebuild on Linux.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants