[Buildroot] [[PATCH v2 03/14] package/mender: create directory structures required by the Mender client

[Mirza Krak]

On Mon, Aug 20, 2018 at 3:43 PM, Mirza Krak <mirza.krak at northern.tech> wrote:
> On Mon, Aug 20, 2018 at 3:35 PM, Mirza Krak <mirza.krak at northern.tech> wrote:
>> On Mon, Aug 20, 2018 at 3:15 PM, Thomas Petazzoni
>> <thomas.petazzoni at bootlin.com> wrote:
>>> Hello,
>>>
>>> On Mon, 20 Aug 2018 15:09:03 +0200, Mirza Krak wrote:
>>>> /uboot is a mount point for vfat boot partition.
>>>> /data is a mount point for the persistent data partition.
>>>>
>>>> Signed-off-by: Mirza Krak <mirza.krak at northern.tech>
>>>> ---
>>>>  package/mender/mender.mk | 4 ++++
>>>>  1 file changed, 4 insertions(+)
>>>>
>>>> diff --git a/package/mender/mender.mk b/package/mender/mender.mk
>>>> index deab0cb039..71f92ce048 100644
>>>> --- a/package/mender/mender.mk
>>>> +++ b/package/mender/mender.mk
>>>> @@ -8,6 +8,10 @@ MENDER_VERSION = 1.4.0
>>>>  MENDER_SITE = $(call github,mendersoftware,mender,$(MENDER_VERSION))
>>>>
>>>>  define MENDER_INSTALL_CONFIG_FILES
>>>> +     $(INSTALL) -d -m 755 $(TARGET_DIR)/data/mender
>>>> +     $(INSTALL) -d -m 755 $(TARGET_DIR)/data/uboot
>>>> +     $(INSTALL) -d -m 755 $(TARGET_DIR)/uboot
>>>
>>> I am sorry, but despite the explanations, I still don't understand why
>>> these *must* be /data and /uboot. Is Mender *enforcing* those locations
>>> exactly ?
>>
>> /uboot, not really enforced, just aligned to other systems that
>> integrate Mender. I could actually drop this.
>>
>> /data is not really enforced but the symlink /var/lib/mender must
>> point to a valid location (normally it points to /data), because once
>> the Mender daemon starts it will generate some initial files at that
>> location and the service would fail if non-existent.
>
> Actually I can drop the /data creation from mender.mk as well, as the
> mender.service will run:
>
>     ExecStartPre=/bin/mkdir -p -m 0700 /data/mender
>
> So yeah a bit enforced.

But what I wanted tot say is that we can probably drop this patch.

-- 
Mirza Krak | Embedded Solutions Architect | https://mender.io

 Northern.tech AS | @northerntechHQ