Skip to contentSkip to conversation

New: Make sure that dest dir exists before installing mtd files

Buildroot bugzilla at busybox.net 1 participant 5 messages Oct 22, 2011 – Nov 2, 2011

lore
Share & tools
lore
#0 lore raw

https://bugs.busybox.net/show_bug.cgi?id=4387

Summary: Make sure that dest dir exists before installing mtd
files
Product: buildroot
Version: unspecified
Platform: PC
OS/Version: Linux
Status: NEW
Severity: minor
Priority: P5
Component: Other
AssignedTo: unassigned at buildroot.uclibc.org
ReportedBy: d_mo1234 at yahoo.com
CC: buildroot at uclibc.org
Estimated Hours: 0.0

Created attachment 3697
--> https://bugs.busybox.net/attachment.cgi?id=3697
patch file

I ran into a situation where a custom target skeleton did not have a /usr/sbin
- it relied on something else to make it (busybox?). So when mtd.mk tried to install things to it, install bombed out. (I think that we were probably using the new version of "make", which no longer sorts wildcards in alphabetical order - so mtd might have been before busybox.)

I have attached a patch to make sure installation directory actually exists. Seems like a prudent thing to do.

Signature · 3 lines
Configure bugmail: https://bugs.busybox.net/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
#1 lore raw

[Buildroot] [Bug 4387] Make sure that dest dir exists before installing mtd files

https://bugs.busybox.net/show_bug.cgi?id=4387

--- Comment #1 from Thomas Petazzoni <[email protected]> 2011-10-31 13:43:08 UTC --- Instead, the patch should change the call to install to use the -D option.
Could you submit a proper patch to the Buildroot mailing list?

Signature · 3 lines
Configure bugmail: https://bugs.busybox.net/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
#2 lore raw

[Buildroot] [Bug 4387] Make sure that dest dir exists before installing mtd files

In reply to bugzilla at busybox.net

https://bugs.busybox.net/show_bug.cgi?id=4387

Danomi Mocelopolis <[email protected]> changed:

What |Removed |Added
----------------------------------------------------------------------------

   Attachment #3697|0                           |1

is obsolete| |

--- Comment #2 from Danomi Mocelopolis <[email protected]> 2011-11-01 01:46:37 UTC --- Comment on attachment 3697
--> https://bugs.busybox.net/attachment.cgi?id=3697
patch file

updated by new file

Signature · 3 lines
Configure bugmail: https://bugs.busybox.net/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
#3 lore raw

[Buildroot] [Bug 4387] Make sure that dest dir exists before installing mtd files

In reply to bugzilla at busybox.net

https://bugs.busybox.net/show_bug.cgi?id=4387

--- Comment #3 from Danomi Mocelopolis <[email protected]> 2011-11-01 01:48:12 UTC --- Created attachment 3716
--> https://bugs.busybox.net/attachment.cgi?id=3716
patch file

Okay, I'll submit to mailing list, if that's preferable to using bugzilla. For completeness, the same file has been updated here.

Signature · 3 lines
Configure bugmail: https://bugs.busybox.net/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
#4 lore raw

[Buildroot] [Bug 4387] Make sure that dest dir exists before installing mtd files

In reply to bugzilla at busybox.net

https://bugs.busybox.net/show_bug.cgi?id=4387

Peter Korsgaard <[email protected]> changed:

What |Removed |Added
----------------------------------------------------------------------------

             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED

--- Comment #4 from Peter Korsgaard <[email protected]> 2011-11-02 11:05:40 UTC --- Committed, thanks

Signature · 3 lines
Configure bugmail: https://bugs.busybox.net/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.