Hi, I have another change to propose to icsv2ledger. Why is there an account mapping file AND a payee mapping file? Why not merging into a single file? The two files have CSV format, first column is regexp, and second is account or payee. What bother me is that I have to change the regexp in both files. I would propose to have a single CSV file with 3 columns regexp,account,payee, (and if needed a variable in config file to use only one column or the other). Thanks for any feedback.
Thierry On Sunday, September 30, 2012 8:08:50 AM UTC+2, Peter Ross wrote: > > On 29 September 2012 22:37, thierry <[email protected] <javascript:>> > wrote: > > Thanks Peter for this message. I re-discovered the tool, and finally > > switched to it! The 'i' of icsv2ledger ('i' meaning interactive) is what > > convinced me. I forked and pulled some changes in github. > > > > I have others changes but they may break some existing functionalities. > I > > write below what I have in mind, that may start a discussion about them: > > - I want to use icsv2ledger through unix pipes. That means ability to > read > > from stdin (I think this is easy implementing this using fileinput > python > > module), and ability to write to stdout. This latter will break existing > way > > of working, as the default is "csv_filename" which csv extension is > replaced > > with .ledger. I procrastinate on how to implement it: 3 different > outputs > > can be used : stdout, csv_filename.ledger or newfilename. But the unix > > philosophy would teach me to use only stdout and newfilename. But this > break > > the existing. And also, this may suppress the ability to treat several > input > > files. Any advise welcomed. > > I think that the best option would be to use "-" as the filename which > represents stdin and when it's "-" then just send the output to stdout > rather than opening a file based on the input filename. > > > - I believe that --no-output-tags and --read-file can be moved to config > > file (instead of executable option) > > > Yes that would be fine as well. >
