<br><br><div class="gmail_quote">On Wed, Dec 10, 2008 at 10:00 AM, Ward Vandewege <span dir="ltr"><<a href="mailto:ward@gnu.org">ward@gnu.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class="Ih2E3d">On Wed, Dec 10, 2008 at 09:50:12AM -0700, Abhishek Kulkarni wrote:<br>
>    openvsa.inc has no OPENVSA_LOG_DIR target, and considering that<br>
>    $(SOURCE_DIR)/$(OPENVSA_TARBALL) creates the LOG_DIR we don't need to add<br>
>    dependency on the LOG_DIR.<br>
<br>
</div>Actually, let's do it the other way around and remove that<br>
<div class="Ih2E3d"><br>
  @ mkdir -p $(OPENVSA_LOG_DIR)<br>
<br>
</div>while keeping the dependency on $(OPENVSA_LOG_DIR).<br>
<br>
That's cleaner. OK?<br></blockquote><div><br>Sure, makes sense.<br><br>Also, most of the main targets already have a dependency on the LOG_DIR targets. Is there a specific reason to add it to the TARBALL_TARGET too?<br>
<br>Corrected patch below:<br><br>Thanks<br>Abhishek<br><br>Index: geodevsa/openvsa.inc<br>===================================================================<br>--- geodevsa/openvsa.inc    (revision 255)<br>+++ geodevsa/openvsa.inc    (working copy)<br>
@@ -19,7 +19,6 @@<br> <br> $(SOURCE_DIR)/$(OPENVSA_TARBALL): | $(OPENVSA_LOG_DIR)<br>     @ mkdir -p $(SOURCE_DIR)/openvsa<br>-    @ mkdir -p $(OPENVSA_LOG_DIR)<br>     @ echo "Fetching openvsa..."<br>     @ $(BIN_DIR)/fetchsvn.sh $(OPENVSA_URL) $(SOURCE_DIR)/openvsa \<br>
     $(OPENVSA_TAG) $@ > $(OPENVSA_FETCH_LOG) 2>&1<br>@@ -31,7 +30,6 @@<br>     @ touch $@<br> <br> $(OPENVSA_SRC_DIR)/vsa_lx.bin: $(OPENVSA_STAMP_DIR)/.unpacked<br>-    @ mkdir -p $(OPENVSA_LOG_DIR)<br>     @ echo "Building openvsa..."<br>
     @(unset LDFLAGS; $(MAKE) -C $(OPENVSA_SRC_DIR) \<br>     > $(OPENVSA_BUILD_LOG) 2>&1)<br>@@ -40,6 +38,9 @@<br>     @ mkdir -p $(shell dirname $(GEODE_UNCOMPRESSED_VSA))<br>     @ cp $< $@<br> <br>+$(OPENVESA_STAMP_DIR) $(OPENVESA_LOG_DIR):<br>
+    @ mkdir -p $@<br>+<br> openvsa: $(OPENVSA_SRC_DIR)/vsa_lx.bin<br> <br> openvsa-clean:<br><br> </div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<br>
<br>
Thanks,<br>
Ward.<br>
<font color="#888888"><br>
--<br>
Ward Vandewege <<a href="mailto:ward@fsf.org">ward@fsf.org</a>><br>
Free Software Foundation - Senior Systems Administrator<br>
</font></blockquote></div><br>