Le 03/07/2014 08:57, Jean-Christophe PLAGNIOL-VILLARD a écrit :
>
> On Jul 3, 2014, at 2:54 PM, Sascha Hauer <s.hauer@xxxxxxxxxxxxxx> wrote:
>
>>
>> On Wed, Jul 02, 2014 at 08:45:28AM +0200, Herve Codina wrote:
>>> Hi,
>>>
>>> I haven't got any news about this patch.
>>> Has it been reviewed ?
>>>
>>> I hope it was not lost in the flow of other patches.
>>> If so, i can resend it.
>>
>> Sorry, I have seen it but then forgot to write an answer.
>>
>>>> diff --git a/include/password.h b/include/password.h
>>>> index 0dd1054..9d84685 100644
>>>> --- a/include/password.h
>>>> +++ b/include/password.h
>>>> @@ -18,8 +18,14 @@
>>>> #ifndef __PASSWORD_H__
>>>> #define __PASSWORD_H__
>>>>
>>>> -#define PASSWD_FILE "/env/etc/passwd"
>>>> -#define PASSWD_DIR "/env/etc/"
>>>> +#ifdef CONFIG_PASSWORD_DIR
>>>> +#define PASSWD_DIR CONFIG_PASSWORD_DIR
>>>> +#else
>>>> +#define PASSWD_DIR "/env/etc"
>>>> +#endif
>>>> +
>>>> +#define PASSWD_FILE PASSWD_DIR"/passwd"
>>
>
> I don’t like this idea to have the passwd file configurable
>
> Why do we need thant?
In my case i have multiple envfs images loaded with loadenv.
and i need to have passwd file from one of these image.
I have /env which is my barebox env and /user which is loaded from envfs
image. I need passwd file in /user/passwd.
In a more general case, the abilily to have passwd file from a directory
other than /env can lead, in a very simple way, to the ability to have
passwd file from a fat filesystem or an ubifs filesystem
I use #define to set this passwd file because i want to set it at
compile time and not at runtime.
If you prefer runtime configuration, i can do the patch according to
sasha suggestion.
Herve
>> I don't like this ifdeffery. If you want to have this configurable
>> please make a
>>
>> static char *passwd_file = "/env/etc/passwd"
>>
>> int set_passwd_file(const char *file)
>> {
>> passwd_file = file;
>> }
>>
>> And call it from your board code.
>>
>> Sascha
>>
>> --
>> Pengutronix e.K. | |
>> Industrial Linux Solutions | http://www.pengutronix.de/ |
>> Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 |
>> Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
>>
>> _______________________________________________
>> barebox mailing list
>> barebox@xxxxxxxxxxxxxxxxxxx
>> http://lists.infradead.org/mailman/listinfo/barebox
>
_______________________________________________
barebox mailing list
barebox@xxxxxxxxxxxxxxxxxxx
http://lists.infradead.org/mailman/listinfo/barebox
