On Fri, Apr 22, 2016 at 12:34:54 +0200, Michal Privoznik wrote:
> On 22.04.2016 11:26, Jiri Denemark wrote:
> > On Fri, Apr 22, 2016 at 11:09:37 +0200, Michal Privoznik wrote:
> >> This is an array, not a hash table. It makes no problem with
> >> newer pythons, but with python2.6 it is a problem:
> >>
> >> File "../../docs/apibuild.py", line 117
> >> "VIR_DEPRECATED", # internal macro to mark deprecated apis
> >> ^
> >> SyntaxError: invalid syntax
> >>
> >> Signed-off-by: Michal Privoznik <mprivozn@xxxxxxxxxx>
> >> ---
> >> docs/apibuild.py | 4 ++--
> >> 1 file changed, 2 insertions(+), 2 deletions(-)
> >>
> >> diff --git a/docs/apibuild.py b/docs/apibuild.py
> >> index 9c82c4a..d065d56 100755
> >> --- a/docs/apibuild.py
> >> +++ b/docs/apibuild.py
> >> @@ -113,10 +113,10 @@ ignored_macros = {
> >> }
> >>
> >> # macros that should be completely skipped
> >> -hidden_macros = {
> >> +hidden_macros = [
> >> "VIR_DEPRECATED", # internal macro to mark deprecated apis
> >> "VIR_EXPORT_VAR", # internal macro to mark exported vars
> >> -}
> >> +]
> >
> > That's another option, but I think the fix Peter pushed for this some
> > time ago is good enough.
> >
>
> I don't think he pushed any fix. What he did in his recent commit in
> this area was to introduce this array. I don't see any other movement in
> that respect. What commit exactly do you have in mind?
Hmm, you're right. I saw "pushed as a build breaker" and thought he had
pushed it already...
That said, ACK, please push your version.
Jirka
--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list