search

hpdeifel / hledger-iadd

A terminal UI as drop-in replacement for hledger add.
BSD 3-Clause "New" or "Revised" License
77 stars 8 forks source link

Update for hledger 1.32 #79

Closed alerque closed 8 months ago

alerque commented 9 months ago

This restriction is blocking Arch Linux from updating hledger.

Is there any reason it can't be relaxed?

https://github.com/hpdeifel/hledger-iadd/blob/11501dd102de0ecff88e60608ef0d92fb8c6c31b/hledger-iadd.cabal#L63C53-L63C53

alerque commented 9 months ago

It does look like some updates and a new release will be needed. I tried just patching out the upper bound and ran into some API differences that affect iadd.

hpdeifel commented 9 months ago

As far as I can see, the API differences just affect the testsuite. Did you encounter any issues with building the main binary? (I'm gonna make a new release regardless. Just curious)

alerque commented 9 months ago

I had not dug that deep, but if that's the case I'm going to build it now. We completely broke the Arch Linux package for this with an unfortunate Haskell snafu in our staging repository. If skipping tests for now is what it takes to get a working package again I'm all for it. Trying now.

alerque commented 9 months ago

That does appear to have worked. The Arch Linux package is unbroken now so thanks for the tip. Of course a release that doesn't need a monkey patch to change the version bounds would be good too, but…

hpdeifel commented 8 months ago

This took longer than it should have, but the release is now out of the door