| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
| |
This makes it safe to compare results to -1 to indicate EOF,
regardless of whether char is considered signed or unsigned;
and so eliminates compiler warnings on platforms such as ARM.
Fixes bug #2058.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
While the current formula works for positive numbers, negative numbers
are incorrectly represented.
One of the issues comes from the fact that floor(x) < x for every x.
`amount_t precision` will always be a non negative number and the code
that attempts to fix the issue for negative number will never run.
If we truncate the number instead, the current formula works for both
positive and negative numbers without making negative numbers a corner
case. So let's do that.
Signed-off-by: Rafael Ascensão <rafa.almas@gmail.com>
|
| |
|
|
|
|
|
|
|
| |
Make it clear that --depth behaves the same for balance and register
reports now.
Fixes: #1952
|
| |
|
|
|
|
| |
Thanks to Richard Kreuter.
|
| |
|
|
|
|
|
| |
- Builds for linux
- Builds for macos
|
|
|
|
|
|
|
|
| |
Handle files that don't end with a new line. Throw an error when the buffer
size is exceeded.
Fixes #516
Contributes to #1149
|
|
|
|
|
| |
To make the file easier to find, move it into the repository root - this
way it is immediately visible when viewing the repo online.
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Problem:
Unable build ledger from source code at Ubuntu 20.04 Focal
./acprep dependencies
will return
acprep: INFO: Invoking primary phase: dependencies
acprep: INFO: Executing phase: dependencies
acprep: INFO: Installing Ledger's build dependencies ...
acprep: INFO: Looks like you are using APT on Ubuntu focal
acprep: INFO: I do not recognize your version of Ubuntu!
Solution:
Add 'focal' to ubuntu-releases
|
|
|
|
|
|
|
|
| |
We were previously relying on an indirect include within Boost headers. We're
using scoped_ptr which is defined in <boost/smart_ptr/scoped_ptr.hpp>.
Bug: https://bugs.gentoo.org/790176
Closes: https://github.com/ledger/ledger/issues/2030
|
|
|
|
| |
Fixes #2035
|
| |
|
|
|
|
|
|
|
| |
Notes were already being escaped. The filename, the transaction code,
the payee,, postings' account names, and postings' cost expressions
were not. Everything else the lisp report prints appears to be a
number or a constant-valued non-string token for Lisp.
|
|
|
|
| |
Fixes #2023.
|
|
|
|
|
| |
An amount may have a (single) leading minus sign, but none after that.
Bug #2001 (and #1809).
|
|
|
|
|
|
|
|
|
|
|
|
| |
Currently the IRC link is broken in the rendered readme page on Github.
Speculating that Github's markdown renderer might not display the text
as a link since it starts with "irc://" instead of "http://" or similar.
Added some text to the bullet point specifying channel and server.
I left the link in-place in case the above issue is resolved at some
point, or if other websites render this README.md file without the
issue.
|
|
|
|
| |
Also fix end-of-command boundary checks for keywords "at", "to", etc.
|
|
|
|
|
|
| |
The "python/server.py" file was removed with fix for #1759.
[ci skip]
|
| |
|
|
|
|
| |
The yamllint config file was incorrect.
|
|
|
|
|
|
|
| |
Windows uses UTF-16 for command line arguments, so use boost::nowide
to convert to UTF-8.
Fixes #1986
|
|
|
|
|
| |
Windows doesn't seem to use UTF-8 by default, so we can specify
encoding='utf-8'. Also, backslashes are confusing to regex parsers.
|
| |
|
|
|
|
| |
Date format %F is not available on all platforms.
|
|
|
|
| |
Useful for building in a MINGW32/64 console on Windows.
|
| |
|
|
|
|
| |
Fixes #1905
|
|
|
|
|
| |
Payees declared on posting's metadata are now validated with `--check-payees`
option. Also, their aliases are now considered on reports as well.
|
| |
|
| |
|
| |
|
|
|
|
|
| |
This changes the error reported when a transaction with no amount is
imported.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Commit 501fbc08ae5493db77bb34f4c4fbe1f3a3bc14e3 changed the behavior
of this function to not return the "equal" result (==0) from the
recursive call. Previously, the function returned the result of the
recursive call unconditionally.
The current behavior causes an assertion error for certain
postings. The regression test added in this commit shows such a
posting.
I found through Travis CI that the old behavior was incomplete and
caused unstable orderings, so reverting to the old behavior doesn't
work. Instead, this change adds a fallback: If the recursive call that
compares the prices numerically returns "equal", then compare the
prices with their original commodity as a tie breaker.
This commit does not change any existing ordering, it only adds
deterministic ordering in a case that currently triggers an assertion
error.
This fixes issue #1998.
|
|
|
|
| |
also add unit tests
|
|
|
|
| |
Fix shellcheck warning SC2034: command appears unused.
Fix completion script if used by zsh with bashcompinit.
|
|
|
|
|
|
|
|
| |
Backported python cmake module expects at least cmake 3.7 (requested
by policy), which will break builds on older cmake versions, like 3.5.
This fix will omit python module inclusion for any cmake version less
than 3.7 and the build will continue without problems.
|
| |
|
|
|
|
|
|
|
|
|
|
|
| |
Ledger is treating balance assignments that have a 0 diff as having
a null posting, leading to the posting being auto-balanced and
therefore causing incorrect values to be returned for the transaction.
I fixed this by just making the posting equal to amt - amt (0 in the
right commodity).
Fixes #1942
|
|
|
|
| |
Fixes #1963
|
|
|
|
| |
Fixed #1954
|
|
|
|
|
|
|
|
|
|
| |
Commit 557ab32 ("Expose a new utility function for balances:
sorted_amounts") split out some code into a separate function.
Unfortunately, in the process an "if" statement was dropped,
leading to amounts being shown that shouldn't be shown because
they are smaller than the display precision.
Fixes #1969
|
| |
|
| |
|
| |
|
| |
|
|
|
| |
Added instructions to install build dependencies in Fedora 32.
|
|
|
|
|
|
|
| |
Document how to pass multiple debug flags to ledger using a regular
expression.
Fixes #1062
|