Command Line Interface

Bug #930789 reported by Michael Nagel
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Simple Scan
Won't Fix
Wishlist
Unassigned

Bug Description

This is a placeholder to discuss a command line interface.

Revision history for this message
Michael Nagel (nailor) wrote :

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.
=========================================

Revision history for this message
Michael Nagel (nailor) wrote :

From IRC (me + Robert Ancell):

(23:43:37) Michael Nagel: there is someone working on a cli interface: https://code.launchpad.net/~holger+lp/simple-scan/simple-scan-cli/+merge/91558
(23:44:07) Michael Nagel: i asked some questions but did not get an answer yet... have you seen it and what do you think about it>
(23:45:18) robert_ancell: yeah, I think I'm in agreement with you there. It doesn't really make too much sense to have a cli mode. You should be able to use scanimage and imagemagick to acheive the same thing on the command line
(23:50:24) Michael Nagel: i will try to ask for another reply from holder, but lets wait for more information. a better way to reproduce some crashes might be worth something, however.
(23:50:50) Michael Nagel: s/holder/holger, the requester/

Personally I do not think there will be a library anytime soon. So either this lives on as a addon/patchset/fork of Simple Scan or Robert considers merging...

Changed in simple-scan:
status: New → In Progress
importance: Undecided → Wishlist
Revision history for this message
Michael Nagel (nailor) wrote :

closing because of lack of feedback

(19:26:27) Michael Nagel: Command Line Interface https://bugs.launchpad.net/simple-scan/+bug/930789
(19:26:39) Michael Nagel: should probably be closed/declined
(19:26:51) robert_ancell: agreed

Changed in simple-scan:
status: In Progress → Won't Fix
Revision history for this message
Holger Hans Peter Freyther (holger+lp) wrote :

Okay, fair enough. Then I will release it separately.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.