Remove large files from /usr/share/cups/ipptool |
|||||||
Issue descriptionThere are a few files in /usr/share/cups/ipptools that are quite large like: -rw-r--r-- 1 root root 1048875 Nov 6 11:35 document-a4.pdf -rw-r--r-- 1 root root 1088024 Nov 6 11:35 document-a4.ps -rw-r--r-- 1 root root 1411293 Nov 6 11:35 document-letter.pdf -rw-r--r-- 1 root root 1466776 Nov 6 11:35 document-letter.ps Is cups directly dependent on these files? Are they like example files? I cannot even open the pdf files since their format is not really pdf. They totally consume about 6MB on rootfs. Is there anyway we can remove them?
,
Nov 7 2017
Yes, they were in a veyron_minnie base image I build.
,
Nov 7 2017
Thanks. We'll look into it. Note for implementer, we should be able to exclude these files during the install.
,
Apr 5 2018
,
Apr 5 2018
,
Apr 5 2018
,
Apr 25 2018
,
Jun 19 2018
I did a little bit of digging and it seems that a few of the "tests" in ipptool make use of these documents. What do we use ipptool for on the system? If we don't use these tests then removing the documents should be fine.
,
Jun 19 2018
Release images don't need ipptool. ipptool is used to test IPP servers. It can be pulled completely from our images.
,
Jun 22 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/3df7a60fa02a3f6dcd74de7def37727d3ca38b4c commit 3df7a60fa02a3f6dcd74de7def37727d3ca38b4c Author: David Valleau <valleau@chromium.org> Date: Fri Jun 22 02:56:22 2018 Patching CUPS to not include test directory This change should free up some space on rootfs that was being taken up by unused test files. BUG= chromium:781919 TEST=Manually tested by emerging and deploying to device Change-Id: Ib195dfba2127eda8ecb913efd43551472c383d24 Reviewed-on: https://chromium-review.googlesource.com/1110480 Commit-Ready: David Valleau <valleau@chromium.org> Tested-by: David Valleau <valleau@chromium.org> Reviewed-by: Chirantan Ekbote <chirantan@chromium.org> [add] https://crrev.com/3df7a60fa02a3f6dcd74de7def37727d3ca38b4c/net-print/cups/files/cups-2.2.2-no-tests.patch [modify] https://crrev.com/3df7a60fa02a3f6dcd74de7def37727d3ca38b4c/net-print/cups/cups-2.1.4.ebuild [rename] https://crrev.com/3df7a60fa02a3f6dcd74de7def37727d3ca38b4c/net-print/cups/cups-2.1.4-r48.ebuild
,
Jun 22 2018
Ended up just removing the entire test directory from the build since we don't use any of the test programs, and the ebuild explicitly "RESTRICTS" the use of these tests since they are not a conventional pass/fail unit test but rather an interactive test program. |
|||||||
►
Sign in to add a comment |
|||||||
Comment 1 by skau@chromium.org
, Nov 6 2017