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

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

[bugzilla at busybox.net]

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.

-- 
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.

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

[bugzilla at busybox.net]

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

--- Comment #1 from Thomas Petazzoni <thomas.petazzoni@free-electrons.com> 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?

-- 
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.

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

[bugzilla at busybox.net]

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

Danomi Mocelopolis <d_mo1234@yahoo.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Attachment #3697|0                           |1
        is obsolete|                            |

--- Comment #2 from Danomi Mocelopolis <d_mo1234@yahoo.com> 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

-- 
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.

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

[bugzilla at busybox.net]

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

--- Comment #3 from Danomi Mocelopolis <d_mo1234@yahoo.com> 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.

-- 
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.

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

[bugzilla at busybox.net]

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

Peter Korsgaard <jacmet@uclibc.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED

--- Comment #4 from Peter Korsgaard <jacmet@uclibc.org> 2011-11-02 11:05:40 UTC ---
Committed, thanks

-- 
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.