Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

bc appears to be required in default configuration #30

Closed
p opened this issue Jul 30, 2021 · 0 comments · Fixed by #32
Closed

bc appears to be required in default configuration #30

p opened this issue Jul 30, 2021 · 0 comments · Fixed by #32

Comments

@p
Copy link
Contributor

p commented Jul 30, 2021

The readme says bc is optional, but it appears to be required in the default configuration:

tinfoil%  ./scan -x 'brother4:bus6;dev1' --no-default-size --mode-hw-default 
Scanning...
scanadf: rounded value of br-x from 211.9 to 211.881
scanadf: rounded value of br-y from 355.6 to 355.567
Scanned document /tmp/scan.G03GKrmbGL/scan-0001
Scanned 1 pages
/home/scan/sane-scan-pdf/scan_perpage: line 81: bc: command not found
Found no scans.

In looking at its invocation in scan_perpage, I think it is always invoked if the scan_perpage script is invoked.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Skip empty page detection if optional bc not available rocketraman/sane-scan-pdf
1 participant
@p