Re: [PATCH 3/8] ls_colors.c: enable coloring on u+x files
- From
Duy Nguyen <pclouds@gmail.com>
- Date
- Mar 20, 2014, 12:14 UTC
- Message-ID
- <CACsJy8DP3buUjT95LwG+Kd70KL67xqEubrHfu9Mqx1RNa-ER2Q@mail.gmail.com>
- In-Reply-To
- <vpqpplhks7n.fsf@anie.imag.fr>
On Thu, Mar 20, 2014 at 6:46 PM, Matthieu Moy <Matthieu.Moy@grenoble-inp.fr> wrote:
Show 7 quoted lines
> Nguyễn Thái Ngọc Duy <pclouds@gmail.com> writes: > >> git-compat-util.h does not seem to carry S_IXUGO. Anyway as far as Git >> is concerned, we only care one executable bit. Hard code it. > > Why not use S_IXUSR instead of a hardcoded value? (already used in > path.c, so shouldn't be a problem wrt portability)
Hmm..maybe cache.h does something to that macro. Will drop this patch and include cache.h.
-- Duy