Patchwork [2/9] builder: Add password for user

login
register
mail settings
Submitter Saul Wold
Date Nov. 14, 2012, 5:32 a.m.
Message ID <ced593109000abdca328d606f02ced75f30a9ca5.1352871028.git.sgw@linux.intel.com>
Download mbox | patch
Permalink /patch/38961/
State New
Headers show

Comments

Saul Wold - Nov. 14, 2012, 5:32 a.m.
This is needed to allow openssh to work correctly for the Eclipse
plugin to have access to the build appliance to view/modify recipes
and lauch builds

Signed-off-by: Saul Wold <sgw@linux.intel.com>
---
 meta/recipes-graphics/builder/builder_0.1.bb |    5 +++--
 1 files changed, 3 insertions(+), 2 deletions(-)
Otavio Salvador - Nov. 14, 2012, 11:21 a.m.
On Wed, Nov 14, 2012 at 3:32 AM, Saul Wold <sgw@linux.intel.com> wrote:

> This is needed to allow openssh to work correctly for the Eclipse
> plugin to have access to the build appliance to view/modify recipes
> and lauch builds
>
> Signed-off-by: Saul Wold <sgw@linux.intel.com>
>

I'd like to be able to override the password used; another point is the
commit log ought to document what the default password and the recipe ought
to have it documented as well.
Mark Hatle - Nov. 14, 2012, 2:09 p.m.
On 11/13/12 11:32 PM, Saul Wold wrote:
> This is needed to allow openssh to work correctly for the Eclipse
> plugin to have access to the build appliance to view/modify recipes
> and lauch builds
>
> Signed-off-by: Saul Wold <sgw@linux.intel.com>
> ---
>   meta/recipes-graphics/builder/builder_0.1.bb |    5 +++--
>   1 files changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/meta/recipes-graphics/builder/builder_0.1.bb b/meta/recipes-graphics/builder/builder_0.1.bb
> index 5c12c14..7ac531c 100644
> --- a/meta/recipes-graphics/builder/builder_0.1.bb
> +++ b/meta/recipes-graphics/builder/builder_0.1.bb
> @@ -1,7 +1,7 @@
>   SUMMARY = "New user to do specific job"
>   DESCRIPTION = "This recipe create a new user named ${PN}, who is used for specific jobs like building. The task can be auto started via mini X"
>   SECTION = "x11"
> -PR = "r4"
> +PR = "r5"
>   LICENSE = "MIT"
>   LIC_FILES_CHKSUM = "file://builder_hob_start.sh;endline=5;md5=84796c3c41785d86100fdabcbdade00e"
>
> @@ -18,7 +18,8 @@ inherit useradd
>   USERADD_PACKAGES = "${PN}"
>   USERADD_PARAM_${PN} = "--system --create-home \
>                          --groups video,tty,audio \
> -                       --user-group ${PN}"
> +                       --user-group ${PN} \
> +                       --password .gLibiNXn0P12"

This should work, but can you add a comment that indicates the plain-text for 
the password?

--Mark

>   do_install () {
>   	install -d -m 755 ${D}${sysconfdir}/mini_x/session.d
>
Saul Wold - Nov. 14, 2012, 4:05 p.m.
On 11/14/2012 06:09 AM, Mark Hatle wrote:
> On 11/13/12 11:32 PM, Saul Wold wrote:
>> This is needed to allow openssh to work correctly for the Eclipse
>> plugin to have access to the build appliance to view/modify recipes
>> and lauch builds
>>
>> Signed-off-by: Saul Wold <sgw@linux.intel.com>
>> ---
>>   meta/recipes-graphics/builder/builder_0.1.bb |    5 +++--
>>   1 files changed, 3 insertions(+), 2 deletions(-)
>>
>> diff --git a/meta/recipes-graphics/builder/builder_0.1.bb
>> b/meta/recipes-graphics/builder/builder_0.1.bb
>> index 5c12c14..7ac531c 100644
>> --- a/meta/recipes-graphics/builder/builder_0.1.bb
>> +++ b/meta/recipes-graphics/builder/builder_0.1.bb
>> @@ -1,7 +1,7 @@
>>   SUMMARY = "New user to do specific job"
>>   DESCRIPTION = "This recipe create a new user named ${PN}, who is
>> used for specific jobs like building. The task can be auto started via
>> mini X"
>>   SECTION = "x11"
>> -PR = "r4"
>> +PR = "r5"
>>   LICENSE = "MIT"
>>   LIC_FILES_CHKSUM =
>> "file://builder_hob_start.sh;endline=5;md5=84796c3c41785d86100fdabcbdade00e"
>>
>>
>> @@ -18,7 +18,8 @@ inherit useradd
>>   USERADD_PACKAGES = "${PN}"
>>   USERADD_PARAM_${PN} = "--system --create-home \
>>                          --groups video,tty,audio \
>> -                       --user-group ${PN}"
>> +                       --user-group ${PN} \
>> +                       --password .gLibiNXn0P12"
>
> This should work, but can you add a comment that indicates the
> plain-text for the password?
>
Ahh, that would take all the fun out!

v2 coming with Otavio's suggestion also!

Sau!

> --Mark
>
>>   do_install () {
>>       install -d -m 755 ${D}${sysconfdir}/mini_x/session.d
>>
>
>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
>
>

Patch

diff --git a/meta/recipes-graphics/builder/builder_0.1.bb b/meta/recipes-graphics/builder/builder_0.1.bb
index 5c12c14..7ac531c 100644
--- a/meta/recipes-graphics/builder/builder_0.1.bb
+++ b/meta/recipes-graphics/builder/builder_0.1.bb
@@ -1,7 +1,7 @@ 
 SUMMARY = "New user to do specific job"
 DESCRIPTION = "This recipe create a new user named ${PN}, who is used for specific jobs like building. The task can be auto started via mini X"
 SECTION = "x11"
-PR = "r4"
+PR = "r5"
 LICENSE = "MIT"
 LIC_FILES_CHKSUM = "file://builder_hob_start.sh;endline=5;md5=84796c3c41785d86100fdabcbdade00e"
 
@@ -18,7 +18,8 @@  inherit useradd
 USERADD_PACKAGES = "${PN}"
 USERADD_PARAM_${PN} = "--system --create-home \
                        --groups video,tty,audio \
-                       --user-group ${PN}"
+                       --user-group ${PN} \
+                       --password .gLibiNXn0P12"
 
 do_install () {
 	install -d -m 755 ${D}${sysconfdir}/mini_x/session.d