svn commit: r365003 - head/devel/elfsh/files

[Alexey Dokuchaev]

On Sun, Aug 17, 2014 at 05:13:47PM +0200, olli hauer wrote:
> On 2014-08-17 16:48, John Marino wrote:
> > patch-mod_foo__main.c is too simple.
> > Try:
> > patch-alpha__bravo__charlie__delta-echo__foxtrot__golf__hotel__india_juliet.cpp
> > 
> > then 20 more of those in the same directory.  Very ugly.
> > 
> > eye of the beholder I guess.
> 
> No, your example looks pretty to me ;)
> 
> At last if I compare your example with '+' or '-' that is also not clash free.
> 
> patch-alpha__bravo__charlie__delta-echo__foxtrot__golf__hotel__india_juliet.cpp
> patch-alpha+bravo+charlie+delta-echo+foxtrot+golf+hotel+india_juliet.cpp
> patch-alpha-bravo-charlie-delta-echo-foxtrot-golf-hotel-india_juliet.cpp
> 
> E.g. in devel/psptoolchain-newlib it is not clear without reading the full
> patch filename in which path the patch applies.

Olli, there always be edge cases that could benefit from a particular path
separator being used instead of the default one (whatever it is).  No one's
preventing you from picking the one that suits better.  The poing is that
most of the times there is no need to uglify patches with __, but currently
it's being used by makepatch.  We don't like it, and more than just talking
about it, there is a working code to play with and see if it makes sense.

./danfe