Subject: Re: bug#26718: Update hexchat to 2.12.4

From: Julien Lepiller <[email-masked]>
Date: Sun, 30 Apr 2017 21:35:44 +0200

Le Sun, 30 Apr 2017 20:19:49 +0200,
Tobias Geerinckx-Rice <[email-masked]> a écrit :

> Julien,
> 
> [...]
> 
> > + (with-directory-excursion "m4"
> > +   (for-each (lambda (f) (delete-file f))
> > +             '("intltool.m4" "libtool.m4" "lt~obsolete.m4"
> > +               "ltoptions.m4" "ltsugar.m4" "ltversion.m4")))
> > + (delete-file-recursively "build-aux")
> > + (delete-file "po/Makefile.in.in")  
> 
> I'd move all of this to a separate snippet.
> These files can cause pain when building manually, too.
What's the policy about using patches, snippet or build phases? I was
recently asked to move code from snippet to build phase because it did
not address a security issue or remove non free or bundled components,
but I can't find anything documenting the choice we should make.