Re: xlist and xoset header files

[Bruno Haible]

Hi Ralf,

Thanks for the report.

Eric Blake wrote:
> An alternative would be adding 'list' to the Depends-on section of xlist 
> (likewise for xoset).  But I'm not sure which of the two approaches is 
> inherently better

The functions defined in gl_xlist.c call functions defined in gl_list.c.
Therefore, adding gl_list.h to the file list would only do half of the job.

Fixed as follows:


2010-01-07  Bruno Haible  <address@hidden>

        xlist, xoset: Fix missing dependency bug, introduced on 2009-12-13.
        * modules/xlist (Depends-on): Add 'list',
        * modules/xoset (Depends-on): Add 'oset'.
        Reported by Ralf Wildenhues <address@hidden>.

*** modules/xlist.orig  Thu Jan  7 21:43:04 2010
--- modules/xlist       Thu Jan  7 21:42:21 2010
***************
*** 7,12 ****
--- 7,13 ----
  m4/gl_list.m4
  
  Depends-on:
+ list
  inline
  stdbool
  xalloc-die
*** modules/xoset.orig  Thu Jan  7 21:43:04 2010
--- modules/xoset       Thu Jan  7 21:42:22 2010
***************
*** 7,12 ****
--- 7,13 ----
  m4/gl_list.m4
  
  Depends-on:
+ oset
  inline
  stdbool
  xalloc-die