Skip to content
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

qtcreator: 4.6.2 -> 4.9.0 #61448

Closed
wants to merge 1 commit into from
Closed

qtcreator: 4.6.2 -> 4.9.0 #61448

wants to merge 1 commit into from

Conversation

SRGOM
Copy link
Member

@SRGOM SRGOM commented May 13, 2019

I am very new to Nix and don't have a lot of experience so let me know if any changes are required. The only testing I have done is on my local machine taking it from 4.6 to 4.9. It compiles fine and runs (for a few minutes so far).

Added myself as a maintainer so the package should stay up to date in future.

Tested on my local machine.
@Lassulus
Copy link
Member

The commit message should state what you updated :)
in this case something like "qtcreator: 4.6 -> 4.9" would be good.

@@ -6,8 +6,8 @@
with stdenv.lib;

let
baseVersion = "4.6";
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It might make sense to do what GNOME packages do: define a single version attribute and split it where needed. It will allow us to us update-source-version program.

url = "mirror://gnome/sources/libdazzle/${stdenv.lib.versions.majorMinor version}/${pname}-${version}.tar.xz";

@c0bw3b c0bw3b changed the title Update to 4.9 qtcreator: 4.6.2 -> 4.9.0 May 13, 2019
@SRGOM SRGOM mentioned this pull request May 13, 2019
10 tasks
@SRGOM
Copy link
Member Author

SRGOM commented May 13, 2019

Sorry about the lousy commit message earlier. I do not have the repo checked out locally so I created #61453 this. Thanks, bye.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants