Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 849590 - media-gfx/sane-backends: broken saned@.service unit file
Summary: media-gfx/sane-backends: broken saned@.service unit file
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal major (vote)
Assignee: No maintainer - Look at https://wiki.gentoo.org/wiki/Project:Proxy_Maintainers if you want to take care of it
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-06-04 02:45 UTC by Daniel Dawson
Modified: 2022-11-04 04:24 UTC (History)
2 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Daniel Dawson 2022-06-04 02:45:29 UTC
It seems there are a couple of problems with saned@.service that cause it to not work:

----
[Unit]
Description=Scanner Service
Requires=saned.socket

[Service]
ExecStart=/usr/sbin/saned
User=scanner
Group=scanner
StandardInput=null
StandardOutput=syslog
StandardError=syslog
Environment=SANE_CONFIG_DIR=@CONFIGDIR@
# If you need to debug your configuration uncomment the next line and
# change it as appropriate to set the desired debug options
# Environment=SANE_DEBUG_DLL=255 SANE_DEBUG_NET=255

[Install]
Also=saned.socket
----

First is the User=. There is no acct-user/scanner package. (Was there ever one? I can't find evidence there was.) There is, however, an acct-user/saned, so maybe that user should be used instead.

The other problem is the "@CONFIGDIR@", which the ebuild does not substitute. Does the value need to be a variable? If not, then perhaps it should just be changed to "/etc/sane.d".
Comment 1 Larry the Git Cow gentoo-dev 2022-11-04 04:19:23 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=3865b4531959596959b487d84fd862e2aac4999f

commit 3865b4531959596959b487d84fd862e2aac4999f
Author:     Sam James <sam@gentoo.org>
AuthorDate: 2022-11-04 04:17:27 +0000
Commit:     Sam James <sam@gentoo.org>
CommitDate: 2022-11-04 04:18:58 +0000

    media-gfx/sane-backends: fix configure w/ clang 16; fix service file & udev
    
    - Fix configure with Clang 16
    - Fix saned_at.service user name (scanner->saned, as scanner is the group)
    - Fix path to setfacl in udev rule (previous one assumed /usr merge)
    - Fix group name in udev rule (saned->scanner)
    - Call udev_reload in pkg_post{rm,inst}
    
    Closes: https://bugs.gentoo.org/876772
    Closes: https://bugs.gentoo.org/859619
    Closes: https://bugs.gentoo.org/849590
    Signed-off-by: Sam James <sam@gentoo.org>

 media-gfx/sane-backends/files/66-saned.rules-r1    |   2 +
 .../sane-backends-1.1.1-configure-clang16.patch    |  43 +++
 .../files/sane-backends-1.1.1-musl.patch           |  34 ++
 media-gfx/sane-backends/files/saned_at.service-r1  |  18 +
 .../sane-backends/sane-backends-1.1.1-r5.ebuild    | 376 +++++++++++++++++++++
 5 files changed, 473 insertions(+)
Comment 2 Sam James archtester Gentoo Infrastructure gentoo-dev Security 2022-11-04 04:24:27 UTC
Thanks!