1 |
-----BEGIN PGP SIGNED MESSAGE----- |
2 |
Hash: SHA1 |
3 |
|
4 |
Sebastian Pipping wrote: |
5 |
> Marijn Schouten (hkBst) wrote: |
6 |
>> Furthermore a lot of our patches are in the sed format and I happen to think |
7 |
>> that's a good thing. |
8 |
> |
9 |
> My current view is that "sed patches" should only be used where |
10 |
> "static" patches don't work, ignoring laziness (including mine) |
11 |
> for the moment. |
12 |
|
13 |
That's enough reason right there. But also, static patches are very often not |
14 |
what I want and they would often break unnecessarily where a sed would not have. |
15 |
Lastly I prefer to have the source changes right there in the ebuild when they |
16 |
are not too elaborate and patches don't allow that. |
17 |
|
18 |
> Why do you feel sed patches are a good thing? Who but the ebuild |
19 |
> writer would prefer that to patches? For instance isn't it much |
20 |
> easier to share patches among distros than parts of very distro- |
21 |
> specific scripts, ebuilds in our case? |
22 |
|
23 |
sed's can very easily be turned into patches when needed, so we don't lose |
24 |
anything. Patches are context dependent and usually this is not why I need. |
25 |
Usually I need to replace certain strings irrespective of how many or where they |
26 |
are or their context and sed is the tool that does exactly this and is more |
27 |
robust to changes in the source that don't matter. |
28 |
|
29 |
Marijn |
30 |
|
31 |
- -- |
32 |
Gods do not want you to think, lest they lose existence. |
33 |
Religions do not want you to think, lest they lose power. |
34 |
|
35 |
Marijn Schouten (hkBst), Gentoo Lisp project, Gentoo ML |
36 |
<http://www.gentoo.org/proj/en/lisp/>, #gentoo-{lisp,ml} on FreeNode |
37 |
-----BEGIN PGP SIGNATURE----- |
38 |
Version: GnuPG v2.0.11 (GNU/Linux) |
39 |
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org |
40 |
|
41 |
iEYEARECAAYFAknI1u4ACgkQp/VmCx0OL2yTzQCgtn3oWQihHbhWvr1/4a8MXncj |
42 |
rgYAnih70WNPw5ErPKf9k7hn22DrUGbS |
43 |
=RNV0 |
44 |
-----END PGP SIGNATURE----- |