Hi,

I posted a bug report about this a month ago, but this community seems 
pretty active so I figured I'd reach out for feedback.

In short, the official build instructions for cygwin 
<https://github.com/ledger/ledger/wiki/Build-instructions-for-Cygwin> don't 
work for ledger 3.1.1 and recent versions of cygwin. I explain why in 
detail in said bug report <http://bugs.ledger-cli.org/show_bug.cgi?id=1166>.

I've made some changes and gotten ledger to compile, (with a simple 23-line 
patch: basically a few extra #ifdefs and a definition for the missing 
strnicmp function) and it appears to work. I'd be happy to update the wiki 
or even submit a pull request with my changes, but I'm afraid my solution 
has a few more issues to work out. Specifically, 6 of the unit tests fail:

The following tests FAILED:
          6 - LedgerMathTests (Failed)
         29 - BaselineTest_cmd-pricedb (Failed)
         31 - BaselineTest_cmd-prices (Failed)
        100 - BaselineTest_opt-datetime-format (Failed)
        197 - BaselineTest_opt-time-report (Failed)
        291 - RegressTest_786A3DD0 (Failed)


I'm hoping someone who knows the code base better than I do can glance at 
this and say something like "Oh, that'll only cause problems if you're 
using these specific features" or "that'll only cause minor cosmetic 
problems" or (if I'm lucky) "Here's an easy fix" or even "Don't worry, 
those tests fail on all platforms, it doesn't mean anything."

Any thoughts?

-- 

--- 
You received this message because you are subscribed to the Google Groups 
"Ledger" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/d/optout.

Reply via email to