Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] patch : dbus  remove existing symbolic link
@ 2009-11-24 10:18 Sagaert Johan
  2009-11-25 10:09 ` Peter Korsgaard
  0 siblings, 1 reply; 2+ messages in thread
From: Sagaert Johan @ 2009-11-24 10:18 UTC (permalink / raw)
  To: buildroot

Hi
 
Building dbus fails if the symbolic link is already present.
this should solve the issue.
 
 
diff -rupN buildroot-2009.11/package/dbus/dbus.mk
buildroot/package/dbus/dbus.mk
--- buildroot-2009.11/package/dbus/dbus.mk 2009-11-23 23:24:09.000000000
+0100
+++ buildroot/package/dbus/dbus.mk 2009-11-24 11:01:44.000000000 +0100
@@ -53,7 +53,8 @@ $(eval $(call AUTOTARGETS,package,dbus))
 
 # fix rebuild if /var/lib is a symlink to /tmp
 $(DBUS_HOOK_POST_BUILD): $(DBUS_TARGET_BUILD)
- rm -rf /tmp/dbus
+ if [  -h $(TARGET_DIR)/var/lib/dbus ]; then \
+  rm -rf $(TARGET_DIR)/var/lib/dbus; fi
  touch $@
 
 $(DBUS_HOOK_POST_INSTALL): $(DBUS_TARGET_INSTALL_TARGET)
 
 
 
I hope this is the right way to submit a patch, don't shoot me if it isn't
 
Johan
 
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20091124/eafdf610/attachment-0001.htm>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: br_dbus.patch
Type: application/octet-stream
Size: 566 bytes
Desc: not available
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20091124/eafdf610/attachment-0001.obj>

^ permalink raw reply	[flat|nested] 2+ messages in thread

* [Buildroot] patch : dbus  remove existing symbolic link
  2009-11-24 10:18 [Buildroot] patch : dbus remove existing symbolic link Sagaert Johan
@ 2009-11-25 10:09 ` Peter Korsgaard
  0 siblings, 0 replies; 2+ messages in thread
From: Peter Korsgaard @ 2009-11-25 10:09 UTC (permalink / raw)
  To: buildroot

>>>>> "Sagaert" == Sagaert Johan <sagaert.johan@skynet.be> writes:

 Sagaert> Hi
 Sagaert> Building dbus fails if the symbolic link is already present.
 Sagaert> this should solve the issue.

Thanks, but there's no need for the symlink check as it will always be
true (we create symlink in post-install hook).

I've committed a slightly reworked patch to git.

-- 
Bye, Peter Korsgaard

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2009-11-25 10:09 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-11-24 10:18 [Buildroot] patch : dbus remove existing symbolic link Sagaert Johan
2009-11-25 10:09 ` Peter Korsgaard

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox