simple-scan-team team mailing list archive

[Michael Nagel <ubuntu@xxxxxxxxxxxxxxxxx>]

There is a merge request: https://code.launchpad.net/~holger+lp/simple-
scan/simple-scan-cli/+merge/91558

Discussion from over there:

=========================================
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
=========================================
shell_wrapper: I'm happy with the PDF generated by simple-scan, after playing with the code (never did any Vala before), I could see myself writing a ncurses UI based on the Book/Page classes, or at least a simple batch mode.

configuration: Sure, I just wondered how to proceed. E.g. if you manage
to provide the lib with stable API/ABI or if you want to include a cli
utility.

unrelated_change: Yes, I don't know how to do git rebase -i with bzr. :)

talk: I'm happy to take the discussion somewhere else.
=========================================

-- 
You received this bug notification because you are a member of Simple
Scan Development Team, which is the registrant for Simple Scan.
https://bugs.launchpad.net/bugs/930789

Title:
  Command Line Interface

Status in Simple Scan:
  New

Bug description:
  This is a placeholder to discuss a command line interface.

To manage notifications about this bug go to:
https://bugs.launchpad.net/simple-scan/+bug/930789/+subscriptions