1 |
On 01/27/2014 09:02 AM, vivo75@×××××.com wrote: |
2 |
> On 01/26/14 23:53, Anthony G. Basile wrote: |
3 |
>> Hi everyone, |
4 |
>> |
5 |
>> A while back, I wrote a python wrapper for install to preserve xattrs. |
6 |
>> Its installed in LIBDIR/portage/bin/install.py. It is *painfully* |
7 |
>> slow. For a package like moodle with 16650 .php files, none of which |
8 |
>> probably need any xattr's set, it takes about 30 mins to install. I |
9 |
>> rewrote the wrapper in C. Replacing the python wrapper with the C |
10 |
>> wrapper, the same example reduces from about 30 mins to 2 mins. |
11 |
>> |
12 |
>> Mike and I did some back and forth about how best to write it. The |
13 |
>> latest version is pretty much done at |
14 |
>> |
15 |
>> https://bugs.gentoo.org/show_bug.cgi?id=465000#c56 |
16 |
>> |
17 |
>> We need to get that integrated into portage. 1) I'm not 100% sure how |
18 |
>> to do that. 2) We may want to install it at /usr/bin/install-xattr |
19 |
>> because I'm sure it will be useful for more than just portage. |
20 |
>> |
21 |
>> Comments? |
22 |
>> |
23 |
> patch install from coreutils (and then upstream changes) is not an option? |
24 |
> they already support selinux contexts |
25 |
> |
26 |
> anyway install-xattr could be useful and /usr/bin would be a good option |
27 |
> IMHO |
28 |
> |
29 |
|
30 |
Been there and I even had a patch ready. Upstream answer was '\0'. The |
31 |
only people who engaged the discussion were gentoo devs. |
32 |
|
33 |
Would patching coreutils have been the better approach? Some people |
34 |
might argue install and cp and mv etc should just copy contents to keep |
35 |
these utilities as simple as possible. Although, as you say, install |
36 |
can copy selinux contexts, and cp can copy xattr attributes. So what's |
37 |
the problem with extending installs functionality to include arbitrary |
38 |
xattr attributes? |
39 |
|
40 |
Anyhow, seeing as upstream is uninterested, I prefer this wrapper to |
41 |
maintaining a local patch against coreutils. |
42 |
|
43 |
|
44 |
-- |
45 |
Anthony G. Basile, Ph. D. |
46 |
Chair of Information Technology |
47 |
D'Youville College |
48 |
Buffalo, NY 14201 |
49 |
(716) 829-8197 |