From mboxrd@z Thu Jan 1 00:00:00 1970 From: Danny Milosavljevic Subject: [PATCH v5] Add hplip Date: Wed, 23 Mar 2016 10:39:36 +0100 Message-ID: <20160323103936.1edb6d18@scratchpost.org> References: <20160312191233.48a661ca@scratchpost.org> <8737ruuj0l.fsf@gnu.org> <20160315210512.23989245@scratchpost.org> <20160315221931.16abe3dc@scratchpost.org> <87vb4mygw4.fsf@igalia.com> <87mvpyyfon.fsf@igalia.com> <20160320011904.50131806@scratchpost.org> <20160321230700.452ca6c1@scratchpost.org> <87r3f2ud9x.fsf@igalia.com> <20160322113001.52af072b@scratchpost.org> <87egb2u6gd.fsf@igalia.com> <20160322201207.790bd384@scratchpost.org> <874mbxshv2.fsf@igalia.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:55692) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aifGW-0004fg-7y for guix-devel@gnu.org; Wed, 23 Mar 2016 05:39:49 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1aifGQ-0001zd-Mg for guix-devel@gnu.org; Wed, 23 Mar 2016 05:39:48 -0400 Received: from dd1012.kasserver.com ([85.13.128.8]:46356) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aifGQ-0001y6-Bh for guix-devel@gnu.org; Wed, 23 Mar 2016 05:39:42 -0400 In-Reply-To: <874mbxshv2.fsf@igalia.com> List-Id: "Development of GNU Guix and the GNU System distribution." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org Sender: guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org To: Andy Wingo Cc: guix-devel@gnu.org Hi, thanks for the review! Here's a new version; it also removes the installer of the proprietary plugin installer. diff --git a/gnu/packages/cups.scm b/gnu/packages/cups.scm index ff3d152..bd85974 100644 --- a/gnu/packages/cups.scm +++ b/gnu/packages/cups.scm @@ -26,6 +26,10 @@ #:use-module (gnu packages) #:use-module (gnu packages avahi) #:use-module (gnu packages compression) + #:use-module (gnu packages libusb) + #:use-module (gnu packages autotools) + #:use-module (gnu packages python) + #:use-module (gnu packages scanner) #:use-module (gnu packages image) #:use-module (gnu packages fonts) ;font-dejavu #:use-module (gnu packages fontutils) @@ -297,3 +301,107 @@ device-specific programs to convert and print many types of files.") ("gnutls" ,gnutls) ("cups-filters" ,cups-filters) ("zlib" ,zlib))))) + +(define-public hplip + (package + (name "hplip") + (version "3.16.2") + (source (origin + (method url-fetch) + (uri (string-append "mirror://sourceforge/hplip/" + "/hplip-" version ".tar.gz")) + (sha256 + (base32 + "1nflgrbyl0fz35djnkn7qsfr5g4sh8lixqna9jvs52wasjllbj7j")))) + (build-system gnu-build-system) + (home-page "http://hplipopensource.com/") + (synopsis "HP Printer Drivers") + (description "Hewlett-Packard Printer Drivers and PPDs.") + ;; Note: X11 license is only used in install-sh. + (license (list license:gpl2 license:bsd-3 license:expat license:x11)) + ;; TODO install apparmor profile files eventually. + (arguments + `(#:configure-flags + `("--disable-network-build" + ,(string-append "--prefix=" (assoc-ref %outputs "out")) + ,(string-append "--sysconfdir=" (assoc-ref %outputs "out") "/etc") + ;; Disable until mime.types merging works (FIXME) + "--disable-fax-build" + "--enable-hpcups-install" + "--enable-new-hpcups" + "--enable-cups_ppd_install" + "--enable-cups_drv_install" + ; TODO add foomatic drv install eventually + ; TODO --enable-policykit eventually + ,(string-append "--with-cupsfilterdir=" + (assoc-ref %outputs "out") + "/lib/cups/filter") + ,(string-append "--with-cupsbackenddir=" + (assoc-ref %outputs "out") + "/lib/cups/backend") + ,(string-append "--with-icondir=" + (assoc-ref %outputs "out") + "/share/applications") + ,(string-append "--with-systraydir=" + (assoc-ref %outputs "out") + "/etc/xdg")) + #:phases (modify-phases %standard-phases + (add-after 'unpack 'fix-libusb + (lambda* (#:key inputs outputs #:allow-other-keys) + (let ((out (assoc-ref outputs "out")) + ;; FIXME use merged ppds (I think actually only drvs need to be merged). + (cupsdir (assoc-ref inputs "cups-minimal"))) + (substitute* "base/g.py" + (("'/usr/share;[^']*'") + (string-append "'" cupsdir "/share'")) + (("'/etc/hp/hplip.conf'") + (string-append "'" out + "/etc/hp/hplip.conf" "'"))) + (substitute* "Makefile.am" + (("[[:blank:]]check-plugin\\.py[[:blank:]]") " ") + ;; FIXME beginning-of-word + (("[[:blank:]]plugin\\.py[[:blank:]]") " ") + (("/usr/include/libusb-1.0") + (string-append (assoc-ref inputs "libusb") + "/include/libusb-1.0")) + (("hplip_statedir[ ]*=[ ]*/var/lib/hp") + ;; only for the installer. + (string-append "hplip_statedir = " out + "/var/lib/hp")) + (("hplip_confdir[ ]*=[ ]*/etc/hp") + ;; only for installing the default config + (string-append "hplip_confdir = " out + "/etc/hp")) + (("halpredir[ ]*=[ ]*/usr/share/hal/fdi/preprobe/10osvendor") + ;; I don't think we use hal + (string-append "halpredir = " out + "/share/hal/fdi/preprobe/10osvendor")) + (("rulesdir[ ]*=[ ]*/etc/udev/rules.d") + ;; udev rules will be merged by base service + (string-append "rulesdir = " out + "/lib/udev/rules.d")) + (("rulessystemdir=/usr/lib/systemd/system") + ;; dummy + (string-append "rulessystemdir = " out + "/lib/systemd/system")) + (("/etc/sane.d") + (string-append out + "/etc/sane.d")))))) + (add-after 'fix-libusb 'autoreconf + (lambda _ + (setenv "AUTOMAKE" "automake --foreign") + (zero? (system* "autoreconf" "-fi"))))))) + ;; Python3 support is available starting from HPLIP-3.15.2 . + (inputs `(("libjpeg" ,libjpeg) + ("cups-minimal" ,cups-minimal) + ("libusb" ,libusb) + ("sane-backends" ,sane-backends) + ("dbus" ,dbus) + ("python-wrapper" ,python-wrapper) + ("python" ,python) + ;; TODO: Make hp-setup find python-dbus. + ("python-dbus" ,python-dbus))) + (native-inputs `(("pkg-config" ,pkg-config) + ("automake" ,automake) + ("autoconf" ,autoconf) + ("libtool" ,libtool)))))