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
vym: 2.6.11 -> 2.7.0 #73483
vym: 2.6.11 -> 2.7.0 #73483
Conversation
Added some fixups I've apparently been sitting on since June? Anyway the first installs the vym manpage, the second fixes some strange installation into $out/vym as well as patches around hardcoded references to As for the PR before those commits: please modify the commit message to adhere to the guidelines... the PR title is the right text :). I didn't make this change to avoid force-pushing to your branch w/o discussion at least :). |
Sorry about that, I submitted this PR quickly through the website and I'm not sure how to change a commit message via the web interface. |
I don't either, but since I'm already setup to push to the branch it's easy enough to do locally :). |
Let's give @AndersonTorres a chance to take a look, but otherwise LGTM. Thanks! |
Please add also |
No need for worrying about wrapping ourselves, that's why this uses mkDerivation from the qt set. From the manual you linked:
On the other hand,
That's good to know, and we should probably use the version upstream marks ready and stable :). If you don't mind, can you link or point me to where they say 2.7.1 isn't ready? Sorry for missing it! On a whim checked repology.org stats and looks like I'm not the only one who missed the memo: |
Ok. I'm sorry, if I caused any hassle.
Concerning that I only have a screenshot of the release notes window, which pops up after starting vym the first time: |
On Sun, 17 Nov 2019 09:46:33 -0800, rys ostrovid ***@***.***> wrote:
> > Please add also `wrapQtAppsHook` in `nativeBuildInputs` or the executable probably won't run properly. The underlying issue is described in [this manual section](https://hydra.nixos.org/build/96804884/download/1/nixpkgs/manual.html#sec-language-qt).
>
> No need for worrying about wrapping ourselves, that's why this uses mkDerivation from the qt set.
>
> From the manual you linked:
>
> > . If you cannot use mkDerivation or mkDerivationWith above, include wrapQtAppsHook
Ok. I'm sorry, if I caused any hassle.
Not at all, we're all trying to ensure this works well for everyone :).
Concerning that I only have a screenshot of the release notes window, which pops up after starting vym the first time:
![vym_unstable](https://user-images.githubusercontent.com/1859691/69011417-6db54100-096a-11ea-869b-e78851a3330c.png)
!!
Wow--so I grabbed the update in June
(eep sorry for not seeing it into a PR myself, and sooner!)
so I very likely completely forgot about this message.
I poked at the code repository, and indeed v2.7.0 is tagged
but there's no 2.7.1 tag?? (what code is in that tarball?! haha)
Thanks for bringing this up-- sounds like we should be on 2.7.0!
…
--
You are receiving this because you commented.
Reply to this email directly or view it on GitHub:
#73483 (comment) part: text/html
|
I should add that when choosing 2.7.1 I just picked the latest version from sourceforge. There was no indication that it was a development version (and there is a folder for development releases that this was not in). I noticed that 2.7.1 only comes as a tarball without releases for Windows, so I assumed it was a minor version that contained a Linux-specific bug fix. Also in the "About" window, the version is listed as 2.7.500 - 2019-05-08 I'm fine switching to whatever version we can agree is the latest stable. |
I would also suggest using version 2.7.0 for packaging. |
There was some concern that the release tarball 2.7.1 actually contains an in-development snapshot rather than a full release. The consensus seems to be that 2.7.0 should be used 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.
Result of nix-review pr 73483
1
1 package were build:
- vym
$ ./results/vym/bin/vym --version
VYM - View Your Mind (c) 2004-2019 Uwe Drechsel
Version: 2.7.0
Build date: 2019-04-14
Codename: Production release
Motivation for this change
Version bump.
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @AndersonTorres