[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [Libguestfs] [PATCH 00/13] syntax-check



Hi Hu,

On Tuesday 23 September 2014 17:20:26 Hu Tao wrote:
> This series includes patches to make `make syntax-check` pass.

Thanks for your work. I have few general notes about it:

- please be more descriptive about each issue: just tell in the first
  line of the commit what is the actual change (like "remove extra
  getopt.h includes"), mentioning the check name in the long
  description. This way it is more clear what the change was about, also
  in case sometime in the future some of those checks is removed.

- in case of unused includes, I'd say it should be fine to just remove
  them together in a single commit

I'll comment on specific issues in each commit.

> Some of the fix require change to maint.mk, but the file is not in git
> repo. Is it intended?

This file comes from gnulib (see .gnulib/top/maint.mk), so changes to it 
need to go to gnulib (and most probably require FSF copyright 
assignment, but I'm not sure about it).

Thanks,
-- 
Pino Toscano


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]