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
Update Aqbanking to 6.1.1 #81176
Update Aqbanking to 6.1.1 #81176
Conversation
@GrahamcOfBorg build aqbanking |
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.
Haven't tested this, but LGTM.
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: |
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.
👍
Added 2 questions
inherit sha256; | ||
}; | ||
|
||
postPatch = '' | ||
sed -i '/^set_and_check(AQBANKING_INCLUDE_DIRS "@aqbanking_headerdir@")/i set_and_check(includedir "@includedir@")' aqbanking-config.cmake.in |
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.
A comment for why that's needed / why it's right would be good.
@@ -1,12 +1,11 @@ | |||
# This file is autogenerated from update.sh in the same directory. |
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.
Is it intended that the update.sh
script is removed below?
If yes, it would be good to write the rationale into the commit message (I'm not familiar with aqbanking
but I'd imagine that script was for convenience to not have to figure out the versions by hand).
Thanks for your comments, I have added comments for both points. |
also update download url Remove update script, because upstream has changed its CMS, which changed the paths of the website, so the update script does not work any more.
also update download url
Thanks! I've force-pushed a small typo fix Merging. |
https://www.aquamaniac.de/rdm/attachments/download/243/aqbanking-6.0.2.tar.gz Seems like the releaseId should be 273 not 243. Although that seems to have a different hash, as well. Any idea what might have happened here? |
@clkamp Could you have a look? |
I will check it |
It seems like all download paths have changed, I will prepare another PR to fix this, but it may take a few days |
Fix in #83423. |
Motivation for this change
This uses the newest available releases, so that connecting to german banks works than with PSD2. Also the download urls are updated to the current place.
This PR is based on #71304, but uses more up to date versions of the packages. After merging this, #68880 can be closed, because gnucash is already at a recent enough version.
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)