Bug 17548 - univention-printserver-pdf verwendet /var/cache statt /var/spool
univention-printserver-pdf verwendet /var/cache statt /var/spool
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: Printserver
UCS 2.3
Other Linux
: P5 normal (vote)
: UCS 3.1
Assigned To: Philipp Hahn
Felix Botner
: interim-1
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2010-02-03 09:37 CET by Philipp Hahn
Modified: 2012-12-12 21:10 CET (History)
2 users (show)

See Also:
What kind of report is it?: ---
What type of bug is this?: ---
Who will be affected by this bug?: ---
How will those affected feel about the bug?: ---
User Pain:
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional):
Max CVSS v3 score:
hahn: Patch_Available+


Attachments
s/cache/spool/ + Samba Share für cups-pdf (1.87 KB, patch)
2010-02-03 09:37 CET, Philipp Hahn
Details | Diff
Fix für univentionPrinterQuotaSupport test in cups-printers.py (473 bytes, patch)
2012-09-03 20:26 CEST, Arvid Requate
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Philipp Hahn univentionstaff 2010-02-03 09:37:09 CET
Created attachment 2269 [details]
s/cache/spool/ + Samba Share für cups-pdf

cups-pdf legt seine Daten normalerweise in /var/spool/cups-pdf/* ab,
univention-printserver-pdf setzt das aber auf /var/cache/cups-pdf/* um.

Ist die UCR-Variable cups/cups-pdf/directory bereits vor dem Join auf einen
anderen Pfad gesetzt, wird in 79univention-printserver-pdf.inst das Samba-Share
mit der falschen Pfadangabe erzeugt.
Comment 1 Philipp Hahn univentionstaff 2012-08-08 09:20:27 CEST
Das Standardverzeichnis wurde geändert.
Das Handbuch wurde angepasst.

svn34697, univention-printserver_6.0.0-1.556.201208072231

ChangeLog: svn14242
\item The direcotry for new installations of the PDF printer was changed from \ucsFile{/var/cache/cups-pdf/} to \ucsFile{/var/spool/cups-pdf/} (\ucsBug{17548}).
Comment 2 Arvid Requate univentionstaff 2012-09-03 20:26:28 CEST
Created attachment 4636 [details]
Fix für univentionPrinterQuotaSupport test in cups-printers.py

Seit commit 34697 kann man mit univention-printserver keine Quota mehr aktivieren. IMHO ist der commit zu umfangreich für das Thema dieses Bugs.


arequate@lagra:~/dev/branches/ucs-3.1/ucs/services/univention-printserver$ svn diff -c34697 | diffstat
 79univention-printserver-pdf.inst                               |   19 
 conffiles/etc/cups/cups-pdf.conf                                |   46 -
 conffiles/etc/pam.d/cups                                        |    2 
 cups-pdf.py                                                     |   10 
 cups-printers.py                                                |  235 ++++------
 debian/changelog                                                |   69 +-
 debian/univention-printserver-pdf.dirs                          |    4 
 debian/univention-printserver-pdf.postinst                      |    9 
 modules/univention/management/console/handlers/cups/_job.py     |    2 
 modules/univention/management/console/handlers/cups/_printer.py |    6 
 modules/univention/management/console/handlers/cups/pykota.py   |   51 +-
 modules/univention/management/console/handlers/cups/tools.py    |    6 
 ppds.py                                                         |   14 
 umc/js/_printers/DetailPage.js                                  |   70 +-
 umc/js/_printers/QuotaDialog.js                                 |   44 -
 umc/js/printers.js                                              |   40 -
 umc/python/printers/__init__.py                                 |  150 +++---
 univention-lpadmin                                              |   14 
 18 files changed, 397 insertions(+), 394 deletions(-)
Comment 3 Philipp Hahn univentionstaff 2012-09-06 10:44:04 CEST
(In reply to comment #2)
> Created an attachment (id=4636) [details]
> Fix für univentionPrinterQuotaSupport test in cups-printers.py
Eingebaut.

svn35345, univention-printserver_6.0.6-1.568.201209060924
ChangeLog: ±0

> IMHO ist der commit zu umfangreich für das Thema dieses Bugs.
...
>  18 files changed, 397 insertions(+), 394 deletions(-)

Filtert man mal die ganzen Änderungen bezüglich Einrückung und Leerzeichen-am-Zeilenende heraus, ist es nur noch ⅓ so wild; den Großteil mach die Umstellung von has_key() auf .get() bzw. is_true() aus; was ucslint eben so gefunden hat.

$ git diff --stat --ignore-space-at-eol --ignore-space-change --ignore-all-space b6bba1da47eb8430d286fa3c7ee90f53137173bc{~1,}
 .../79univention-printserver-pdf.inst              |   17 ++-
 .../conffiles/etc/cups/cups-pdf.conf               |    2 +-
 .../conffiles/etc/pam.d/cups                       |    2 +-
 .../services/univention-printserver/cups-pdf.py    |   10 +-
 .../univention-printserver/cups-printers.py        |  147 ++++++++++----------
 .../univention-printserver/debian/changelog        |    9 +-
 .../debian/univention-printserver-pdf.dirs         |    4 +-
 .../debian/univention-printserver-pdf.postinst     |    7 +-
 .../management/console/handlers/cups/_job.py       |    2 +-
 .../management/console/handlers/cups/_printer.py   |    6 +-
 .../management/console/handlers/cups/pykota.py     |   47 +++---
 .../management/console/handlers/cups/tools.py      |    6 +-
 .../ucs/services/univention-printserver/ppds.py    |    8 +-
 .../umc/python/printers/__init__.py                |    2 +-
 .../univention-printserver/univention-lpadmin      |    8 +-
 15 files changed, 140 insertions(+), 137 deletions(-)
Comment 4 Felix Botner univentionstaff 2012-09-13 16:18:31 CEST
PDF Drucker kann angelegt werden, es kann gedruckt werden, das Share wird eingerichtet und funktioniert (beinhaltet die PDF's).

DruckerQuota kann aktiviert werden.

In UCS@school wird nur die URCV cups/cups-pdf/directory verwendet, nicht der alte default Pfad /var/cache/cups-pdf/

Beim Update wird cups/cups-pdf/directory beibehalten.

Im Join Script wird die Freigabe nun mit dem Inhalt aus cups/cups-pdf/directory (default /var/cache/cups-pdf/) als Pfad angelegt.

Changelog OK
Comment 5 Stefan Gohmann univentionstaff 2012-12-12 21:10:24 CET
UCS 3.1-0 has been released: 
 http://forum.univention.de/viewtopic.php?f=54&t=2125

If this error occurs again, please use "Clone This Bug".