-
-
Notifications
You must be signed in to change notification settings - Fork 14.8k
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
qgis: 2.18.28 -> 3.4.5 #56486
qgis: 2.18.28 -> 3.4.5 #56486
Conversation
@lsix could you rebase onto master and check |
@FRidh I have rebased onto master. I’ll check ASAP what is happening with qscintilla (qscintilla-qt5 is fixed as part of this PR). |
@FRidh It seems that I have a new problem for qscintilla-qt5 after rebase. I’ll let you know as soon as I fix it. |
@FRidh I have not had much time to look into what changed recently, but my patches works fine against master from 3 days ago and completely fails on a recent master (mainly I’ll try to have a proper look at it this week-end |
There was a Qt upgrade, qt511: 5.11.1 -> 5.11.3, and I fixed pyqt5 in 6bffe97 |
If I check out branch nix run -f /path/to/nixpkgs qgis -c qgis If I check out branch
If I install QGIS 3, e.g. with nix-env -i /nix/store/5dgl07kvbvm1s5n07dxsv6ccjfhx95q0-qgis-3.4.5 then I can start it. (As far as I remember, I already got the same error message with the branch from the older, now abandoned and closed PR #38022.) Is this expected? Output of
|
@FRidh I have pushed a rebased, fixed and working version of the PR. For the moment, I have disabled the dependency on @das-g I am experiencing the same thing, it is, as far as I know, related to qt5 (qt-2.18 was Qt4 based). |
the build jobs are failing because of timeout, but this is not unexpected given the time required to build QGis. On aarch64-linux, the failure is related to qtwebkit, so unrelated to this PR. |
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.
Seems like the GRASS description has been changed instead of the QGIS description.
fd285d4
to
716fd94
Compare
@das-g while I was at it, I added a |
As far as I can see, their |
done. I have used meta = qgis-unwrapped.meta; instead |
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.
Looks good to me.
(For whatever that's worth. I still consider myself a nixpkgs noob.)
Hi,
To do that, I just need to rebase, and remove the |
Motivation for this change
QGis-3.4.5 is the new LTR version.
We should use this instead of the now obselete 2.18 one for the upcoming 19.03 release. I am really sorry this comes this close to the branch off deadline.
See #35669
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)