-
-
Notifications
You must be signed in to change notification settings - Fork 14k
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
qt5.qutebrowser: reinit at 3.0.2 #265364
qt5.qutebrowser: reinit at 3.0.2 #265364
Conversation
@ofborg build qt5.qutebrowser |
@ofborg eval |
What's wrong with #251671? That looks much simpler. |
readability-lxml pykeepass stem | ||
readability-lxml pykeepass | ||
] ++ lib.optionals ((builtins.tryEval stem.outPath).success) [ | ||
# error: stem-1.8.2 not supported for interpreter python3.11 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That was fixed by #262813.
sorry but I don't think its considered vandalism to not support a browser backend which is severely outdated and probaby rife with security vulnerabilities at this point. It was a conscious decision and one recommended by upstream. In any case, I don't think it's a good idea to support this, but if you really want to, you should probably have some sort of warning during eval that the browser backend is the insecure. |
Now this is getting ridiculous. I don't really understand how could anyone get so worked up on something so trivial, but please drop this. As multiple persons have already told you, it didn't make sense to support the old backend active as it is both outdated (even basic web apps such as discourse refuse to work) and insecure. Also, having two variants of qutebrowser in the same expression made it more complicated and it was a hassle to maintain. IIRC you want to keep Qt5 because Qt6 does not cross compile, right? I would tell you to work on fixing Qt6 instead, but I guess it's ok to keep a copy of the package with Qt pinned at Qt 5.15, like it's usually done for libraries like boost, etc. Downgrading the default package, however it's not an option. Anyway, I don't understand the purpose of this PR considering that #251671 does exactly the same thing, sans the bit about the |
These commits were merged in #264965 without making requested changes. |
@amjoseph-nixpkgs nixpkgs is a shared resource, and we have to find ways to work together. For this to work it's important that:
If there are things frustrating you then let's find ways to make it smoother. Please DM me if you would like to discuss this privately. |
Description of changes
Our qutebrowser package was vandalized by various commits, such as c9cc3a2 and ad0bbaf, which made erroneous assertions such as "since qutebrowser 3.0.0 the derivation is only building for qt6."
This commit repairs the vandalism.
Things done
nix.conf
? (See Nix manual)sandbox = relaxed
sandbox = true
nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)