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.

Sign up for GitHub

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

Jump to bottom

kmymoney: 5.0.5 -> 5.0.7 #69616

Closed
wants to merge 1 commit into from
Closed

kmymoney: 5.0.5 -> 5.0.7 #69616

wants to merge 1 commit into from
+2 −2

Conversation

r-ryantm
Copy link
Contributor

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/kmymoney/versions.

meta.description for kmymoney is: '"Personal finance manager for KDE"'.

meta.homepage for kmymoney is: '"https://kmymoney.org/"

Checks done (click to expand)
Rebuild report (if merged into master) (click to expand)

3 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
0 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
kmymoney

Instructions to test this update (click to expand)

Either download from Cachix:

nix-store -r /nix/store/qy9fyhjdp70h4rnds6z8gqzlcc8q5j3j-kmymoney-5.0.7 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)

Or, build yourself:

nix-build -A kmymoney https://github.com/r-ryantm/nixpkgs/archive/43f21201425c8728a1c067529069237fe1d24b54.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:

ls -la /nix/store/qy9fyhjdp70h4rnds6z8gqzlcc8q5j3j-kmymoney-5.0.7
ls -la /nix/store/qy9fyhjdp70h4rnds6z8gqzlcc8q5j3j-kmymoney-5.0.7/bin

@r-ryantm
kmymoney: 5.0.5 -> 5.0.7
43f2120 
Semi-automatic update generated by
https://github.com/ryantm/nixpkgs-update tools. This update was made
based on information from
https://repology.org/metapackage/kmymoney/versions
@ofborg ofborg bot added 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin 10.rebuild-linux: 1-10 labels Sep 27, 2019
@aanderse aanderse mentioned this pull request Sep 27, 2019
Closed
@thorstenweber83 thorstenweber83 mentioned this pull request Oct 18, 2019
Closed
10 tasks
thorstenweber83
thorstenweber83 approved these changes Oct 18, 2019
View reviewed changes
Copy link
Contributor

@thorstenweber83 thorstenweber83 left a comment

Choose a reason for hiding this comment

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

Tested the binary and it looks good from what i've seen!

das-g
das-g approved these changes Oct 20, 2019
View reviewed changes
Copy link
Member

@das-g das-g left a comment

Choose a reason for hiding this comment

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

Seems to work fine.

@das-g
Copy link
Member

das-g commented Oct 21, 2019

👍 This seems to also fix error

Scanning dependencies of target interfaces
[  6%] Building CXX object kmymoney/plugins/interfaces/CMakeFiles/interfaces.dir/interfaces_autogen/mocs_compilation.cpp.o
In file included from /build/kmymoney-5.0.5/build/kmymoney/plugins/kmm_plugin_autogen/EWIEGA46WW/moc_kmymoneyplugin.cpp:9,
                 from /build/kmymoney-5.0.5/build/kmymoney/plugins/kmm_plugin_autogen/mocs_compilation.cpp:4:
/build/kmymoney-5.0.5/build/kmymoney/plugins/kmm_plugin_autogen/EWIEGA46WW/../../../../../kmymoney/plugins/kmymoneyplugin.h:358:40: error: field 'standard' has incomplete type 'QMap<QString, KMyMoneyPlugin::Plugin*>'
   QMap<QString, Plugin*>               standard;  // this should contain all loaded plugins because every plugin should inherit Plugin class
                                        ^~~~~~~~
In file included from /nix/store/2i8a07msylyph4l6ksdd1ixdbh17rq5r-qtbase-5.12.3-dev/include/QtCore/qglobal.h:1204,
                 from /nix/store/2i8a07msylyph4l6ksdd1ixdbh17rq5r-qtbase-5.12.3-dev/include/QtCore/qnamespace.h:43,
                 from /nix/store/2i8a07msylyph4l6ksdd1ixdbh17rq5r-qtbase-5.12.3-dev/include/QtCore/qobjectdefs.h:48,
                 from /nix/store/2i8a07msylyph4l6ksdd1ixdbh17rq5r-qtbase-5.12.3-dev/include/QtCore/qobject.h:46,
                 from /nix/store/2i8a07msylyph4l6ksdd1ixdbh17rq5r-qtbase-5.12.3-dev/include/QtCore/QObject:1,
                 from /build/kmymoney-5.0.5/build/kmymoney/plugins/kmm_plugin_autogen/EWIEGA46WW/../../../../../kmymoney/plugins/appinterface.h:23,
                 from /build/kmymoney-5.0.5/build/kmymoney/plugins/kmm_plugin_autogen/EWIEGA46WW/moc_appinterface.cpp:9,
                 from /build/kmymoney-5.0.5/build/kmymoney/plugins/kmm_plugin_autogen/mocs_compilation.cpp:2:
/nix/store/2i8a07msylyph4l6ksdd1ixdbh17rq5r-qtbase-5.12.3-dev/include/QtCore/qtypeinfo.h:221:1: note: declaration of 'class QMap<QString, KMyMoneyPlugin::Plugin*>'
 Q_DECLARE_MOVABLE_CONTAINER(QMap);
 ^~~~~~~~~~~~~~~~~~~~~~~~~~~

(and similar errors for other fields) that got kmymoney marked as broken as part of #68361.

Thus this change should probably be backported to release-19.09.

das-g added a commit to das-g/nixpkgs that referenced this pull request Oct 21, 2019
@das-g
kmymoney: 5.0.5 -> 5.0.7
5383748 
and mark as not broken anymore

see PR NixOS#69616 for the corresponding change on master
@das-g das-g mentioned this pull request Oct 21, 2019
Merged
12 tasks
@globin globin closed this Oct 23, 2019
@r-ryantm r-ryantm deleted the auto-update/kmymoney branch November 1, 2019 13:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@das-g das-g

@thorstenweber83 thorstenweber83

Assignees
No one assigned
Labels
10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin 10.rebuild-linux: 1-10
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@r-ryantm @das-g @thorstenweber83 @globin