Ted Percival wrote:
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?

On Solaris if build-db command succeeds [EMAIL PROTECTED] does not exist but [EMAIL PROTECTED] and [EMAIL PROTECTED] do.

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.


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

Reply via email to