1 |
-----BEGIN PGP SIGNED MESSAGE----- |
2 |
Hash: SHA512 |
3 |
|
4 |
Andrew Savchenko: |
5 |
> On Sun, 24 Jan 2016 16:00:31 +0100 Patrice Clement wrote: |
6 |
>> Sunday 24 Jan 2016 03:46:02, Gokturk Yuksek wrote : |
7 |
>>> Remove the paragraph about adding binary files to CVS. Replace |
8 |
>>> the instances of CVS with git. |
9 |
>>> |
10 |
>>> X-Gentoo-Bug-URL: |
11 |
>>> https://bugs.gentoo.org/show_bug.cgi?id=558642 Signed-off-by: |
12 |
>>> Gokturk Yuksek <gokturk@××××××××××.edu> --- |
13 |
>>> ebuild-maintenance/text.xml | 19 +++---------------- 1 file |
14 |
>>> changed, 3 insertions(+), 16 deletions(-) |
15 |
>>> |
16 |
>>> diff --git a/ebuild-maintenance/text.xml |
17 |
>>> b/ebuild-maintenance/text.xml index ac68dcd..10709ea 100644 |
18 |
>>> --- a/ebuild-maintenance/text.xml +++ |
19 |
>>> b/ebuild-maintenance/text.xml @@ -42,22 +42,9 @@ above 20KB) |
20 |
>>> which should be distributed as tarballs via the <uri |
21 |
>>> link="::general-concepts/mirrors/#suitable-download-hosts">Gentoo |
22 |
>>> |
23 |
>>> |
24 |
mirror system</uri> so that people do not waste excessive |
25 |
>>> amounts of bandwidth and hard drive space. Also, you should |
26 |
>>> not add binary (non-ASCII) files to the -Portage CVS tree. If |
27 |
>>> you need to do this in another CVS tree, for -example, if you |
28 |
>>> need to add a small PNG graphic for whatever reason, -be sure |
29 |
>>> to add it to CVS by using the <c>-kb</c> option, like so: -</p> |
30 |
>>> - -<pre caption="Adding binary files to CVS"> -# <i>cvs add |
31 |
>>> -kb myphoto.png</i> -</pre> - -<p> -The <c>-kb</c> option tells |
32 |
>>> CVS that <path>myphoto.png</path> is a binary -file and should |
33 |
>>> be treated specially. For example, merging the -differences |
34 |
>>> between two different versions of this file should not be |
35 |
>>> -allowed to happen, for obvious reasons. Also, speaking of |
36 |
>>> merging +git tree. Also, speaking of merging changes, any |
37 |
>>> patches you add to Portage should generally <e>not</e> be |
38 |
>>> -compressed. This will allow CVS to merge changes and |
39 |
>>> correctly inform +compressed. This will allow git to merge |
40 |
>>> changes and correctly inform developers of conflicts. </p> |
41 |
>>> |
42 |
>>> @@ -149,7 +136,7 @@ simply, <path>1.0-gentoo.diff</path>. |
43 |
>>> Also note that the <path>gentoo</path> extension informs people |
44 |
>>> that this patch was created by us, the Gentoo Linux |
45 |
>>> developers, rather than having been grabbed from a mailing list |
46 |
>>> or somewhere else. Again, you should not compress these |
47 |
>>> -patches because CVS does not play well with binary files. |
48 |
>>> +patches because git does not play well with binary files. |
49 |
>>> </p> |
50 |
>>> |
51 |
>>> <p> -- 2.4.10 |
52 |
>>> |
53 |
>>> |
54 |
>> |
55 |
>> "Again you should not compress these patches because git does |
56 |
>> not play well binary files". |
57 |
>> |
58 |
>> I'm not sure this statement still holds true with git. Does it? |
59 |
> |
60 |
> What about repoman checks? Will it still yell at >20 kB patches? |
61 |
> |
62 |
Yessir: |
63 |
https://gitweb.gentoo.org/proj/portage.git/tree/pym/repoman/checks/ebuilds/fetches.py?h=v2.2.27#n104 |
64 |
|
65 |
Even though it's not an issue with the git tree, rsync users will |
66 |
still suffer for >20kB patches. |
67 |
|
68 |
> Best regards, Andrew Savchenko |
69 |
> |
70 |
|
71 |
- -- |
72 |
gokturk |
73 |
|
74 |
-----BEGIN PGP SIGNATURE----- |
75 |
|
76 |
iQEcBAEBCgAGBQJWpWG/AAoJEIT4AuXAiM4zL4kIAJOQXCo2veG56lBhFjbYcusi |
77 |
KZwJUbSpzLW6liZYlVYE6lYLsi7m/AW0jNhTWHt4P6vqsyhSzvS+GzchZbPzmzel |
78 |
j9dSurvwVEtrAucLbinQ95Ym8biRVREw6ieC3D+cMZbsd9+yvLdA9vWvIDneOA09 |
79 |
BsTmibtNHXg3xusDhuAK3hl8lxZhu5dK0n733Mm/S3gnDJ952dGyQjIMxZhxdwSd |
80 |
QOK2VTRfUJx0hAoNUPNqd0flOCBfijdOR7qFDZuVNsyqZwTW7Du+k8uz8uRV+hKp |
81 |
4dktGALgDh4tTd6O8kCrQBTBQ2IVC03Fvs3xvAYFV84WNug3tEYRB2mDUpGBzck= |
82 |
=bpVR |
83 |
-----END PGP SIGNATURE----- |