Padraig O'Briain wrote:
> I was able to get it to build with the attached patch.
> 
>  service-types.db: service-types build-db
>       $(PYTHON) build-db $< [EMAIL PROTECTED]
> -     mv [EMAIL PROTECTED] $@
> +     if test -f "[EMAIL PROTECTED]"; then mv [EMAIL PROTECTED] $@; fi
>  

I'm not sure why this is necessary. The change to this bit of code was
made so that $@ would only be created if the build-db command succeeded.
It's possible for the build-db command to fail but the $@ file to be
created, which means if `make` is run again, it will consider the rule
satisfied.

I don't see how adding an 'if' around the mv command helps at all.
Surely the mv command will only be attempted if the build-db command
succeeds, in which case the [EMAIL PROTECTED] file will definitely exist. Have I
missed something?

If this is the right change to make, it might also need to be made near
line 35 where there is a similar service-types.db rule.

-- 
tedp
_______________________________________________
avahi mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/avahi

Reply via email to