Hi Holger, ultimately Robert will have to approve ;)
I have never worked with such merge requests. Here are a few thoughts of mine though: - thanks for contributing! - why didn't you write a three line shell wrapper around scanimage that scans to a temporary file and then calls some combination of convert/pdftk? i am positively interested in this! - what's the benefit from merging this into trunk? it's not exactly in line with the mission statement for simple scan. (there is no mission statement -- but if there was one it would be about making scanning simple for casual users). on the other hand, it might increase usage of the simple-scan, leading to fixes in the code for the benefit of all users. also it might improve maintainability by allowing to easily reproduce crashes... - allowing some configuration via command line arguments would definitly make it more useful for other users... - there seem to be some unrelated changes mixed into the commits, like changing from grayscale to lineart. lineart is theoretically correct, but caused problems for many users, so grayscale it is... Thanks again, it would be great if you could comment on some of these and I am waiting for a comment from Robert. Also, I am not really sure if this is the best place to have such discussion, but let's just try. If it does not work out, we might just open a ticket/bug report, IMHO those can be abused for general discussion quite well... Best Regards -- https://code.launchpad.net/~holger+lp/simple-scan/simple-scan-cli/+merge/91558 Your team Simple Scan Development Team is requested to review the proposed merge of lp:~holger+lp/simple-scan/simple-scan-cli into lp:simple-scan. _______________________________________________ Mailing list: https://launchpad.net/~simple-scan-team Post to : [email protected] Unsubscribe : https://launchpad.net/~simple-scan-team More help : https://help.launchpad.net/ListHelp
