All of lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH] package/pseudo: update version
@ 2016-11-25 21:22 Gaël PORTAY
  2016-11-26 10:50 ` Arnout Vandecappelle
  0 siblings, 1 reply; 7+ messages in thread
From: Gaël PORTAY @ 2016-11-25 21:22 UTC (permalink / raw)
  To: buildroot

Brings in fix about shutdown synchronization with the server.

See commit message:
    If you're running pseudo in docker, a script that creates a pseudo
    daemon can exit, causing docker to kill pseudo before it's done writing
    the database.

    Since the client sending the shutdown request doesn't have its socket
    closed explicitly by the server, we can just read from the socket in
    the client to create a delay until the actual exit, which can take
    a while if there's an in-memory DB.

Furthermore, the patch add the -S argument that tells the server to
shutdown and waits for its completion.

Signed-off-by: Ga?l PORTAY <gael.portay@savoirfairelinux.com>
---
 package/pseudo/pseudo-wrapper | 2 +-
 package/pseudo/pseudo.mk      | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/package/pseudo/pseudo-wrapper b/package/pseudo/pseudo-wrapper
index 9c8dbdb..ad66431 100644
--- a/package/pseudo/pseudo-wrapper
+++ b/package/pseudo/pseudo-wrapper
@@ -16,4 +16,4 @@ if [ -n "${BASE_DIR}" ]; then
     export PSEUDO_LOCALSTATEDIR="${BASE_DIR}/build/.pseudodb"
 fi
 
-exec "${0%/*}/pseudo" "${@}"
+exec "${0%/*}/pseudo" -S "${@}"
diff --git a/package/pseudo/pseudo.mk b/package/pseudo/pseudo.mk
index aa2a9c7..58b9c79 100644
--- a/package/pseudo/pseudo.mk
+++ b/package/pseudo/pseudo.mk
@@ -4,7 +4,7 @@
 #
 ################################################################################
 
-PSEUDO_VERSION = 45eca34c754d416a38bee90fb2d3c110a0b6cc5f
+PSEUDO_VERSION = d6eb2df3fe63b765f35d62332add4d0e4e9c6a39
 PSEUDO_SITE = https://git.yoctoproject.org/git/pseudo
 PSEUDO_SITE_METHOD = git
 
-- 
2.10.2

^ permalink raw reply related	[flat|nested] 7+ messages in thread

* [Buildroot] [PATCH] package/pseudo: update version
  2016-11-25 21:22 [Buildroot] [PATCH] package/pseudo: update version Gaël PORTAY
@ 2016-11-26 10:50 ` Arnout Vandecappelle
  2016-11-26 14:02   ` Thomas Petazzoni
  2016-11-26 16:25   ` Gaël PORTAY
  0 siblings, 2 replies; 7+ messages in thread
From: Arnout Vandecappelle @ 2016-11-26 10:50 UTC (permalink / raw)
  To: buildroot



On 25-11-16 22:22, Ga?l PORTAY wrote:
> Brings in fix about shutdown synchronization with the server.
> 
> See commit message:
>     If you're running pseudo in docker, a script that creates a pseudo
>     daemon can exit, causing docker to kill pseudo before it's done writing
>     the database.
> 
>     Since the client sending the shutdown request doesn't have its socket
>     closed explicitly by the server, we can just read from the socket in
>     the client to create a delay until the actual exit, which can take
>     a while if there's an in-memory DB.
> 
> Furthermore, the patch add the -S argument that tells the server to
> shutdown and waits for its completion.

 Does this work? From the man page, I understand that the -S option can't be
combined with starting a program.

> 
> Signed-off-by: Ga?l PORTAY <gael.portay@savoirfairelinux.com>
> ---
>  package/pseudo/pseudo-wrapper | 2 +-
>  package/pseudo/pseudo.mk      | 2 +-

 Did you test this? You didn't update the hash file...

 Regards,
 Arnout

>  2 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/package/pseudo/pseudo-wrapper b/package/pseudo/pseudo-wrapper
> index 9c8dbdb..ad66431 100644
> --- a/package/pseudo/pseudo-wrapper
> +++ b/package/pseudo/pseudo-wrapper
> @@ -16,4 +16,4 @@ if [ -n "${BASE_DIR}" ]; then
>      export PSEUDO_LOCALSTATEDIR="${BASE_DIR}/build/.pseudodb"
>  fi
>  
> -exec "${0%/*}/pseudo" "${@}"
> +exec "${0%/*}/pseudo" -S "${@}"
> diff --git a/package/pseudo/pseudo.mk b/package/pseudo/pseudo.mk
> index aa2a9c7..58b9c79 100644
> --- a/package/pseudo/pseudo.mk
> +++ b/package/pseudo/pseudo.mk
> @@ -4,7 +4,7 @@
>  #
>  ################################################################################
>  
> -PSEUDO_VERSION = 45eca34c754d416a38bee90fb2d3c110a0b6cc5f
> +PSEUDO_VERSION = d6eb2df3fe63b765f35d62332add4d0e4e9c6a39
>  PSEUDO_SITE = https://git.yoctoproject.org/git/pseudo
>  PSEUDO_SITE_METHOD = git
>  
> 

-- 
Arnout Vandecappelle                          arnout at mind be
Senior Embedded Software Architect            +32-16-286500
Essensium/Mind                                http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium           BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint:  7493 020B C7E3 8618 8DEC 222C 82EB F404 F9AC 0DDF

^ permalink raw reply	[flat|nested] 7+ messages in thread

* [Buildroot] [PATCH] package/pseudo: update version
  2016-11-26 10:50 ` Arnout Vandecappelle
@ 2016-11-26 14:02   ` Thomas Petazzoni
  2016-11-26 15:11     ` Arnout Vandecappelle
  2016-11-26 16:25   ` Gaël PORTAY
  1 sibling, 1 reply; 7+ messages in thread
From: Thomas Petazzoni @ 2016-11-26 14:02 UTC (permalink / raw)
  To: buildroot

Hello,

On Sat, 26 Nov 2016 11:50:45 +0100, Arnout Vandecappelle wrote:

> >  package/pseudo/pseudo-wrapper | 2 +-
> >  package/pseudo/pseudo.mk      | 2 +-  
> 
>  Did you test this? You didn't update the hash file...

Hash files are not checked for git downloads, so it can work without
updating the hash. Since the git download helper was fixed to generate
reproducible archives, we haven't (yet) enabled hash checking for
git downloads.

Best regards,

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

^ permalink raw reply	[flat|nested] 7+ messages in thread

* [Buildroot] [PATCH] package/pseudo: update version
  2016-11-26 14:02   ` Thomas Petazzoni
@ 2016-11-26 15:11     ` Arnout Vandecappelle
  0 siblings, 0 replies; 7+ messages in thread
From: Arnout Vandecappelle @ 2016-11-26 15:11 UTC (permalink / raw)
  To: buildroot



On 26-11-16 15:02, Thomas Petazzoni wrote:
> Hello,
> 
> On Sat, 26 Nov 2016 11:50:45 +0100, Arnout Vandecappelle wrote:
> 
>>>  package/pseudo/pseudo-wrapper | 2 +-
>>>  package/pseudo/pseudo.mk      | 2 +-  
>>
>>  Did you test this? You didn't update the hash file...
> 
> Hash files are not checked for git downloads, so it can work without
> updating the hash. Since the git download helper was fixed to generate
> reproducible archives, we haven't (yet) enabled hash checking for
> git downloads.

 Ah, I never noticed that...

 Regards,
 Arnout

-- 
Arnout Vandecappelle                          arnout at mind be
Senior Embedded Software Architect            +32-16-286500
Essensium/Mind                                http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium           BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint:  7493 020B C7E3 8618 8DEC 222C 82EB F404 F9AC 0DDF

^ permalink raw reply	[flat|nested] 7+ messages in thread

* [Buildroot] [PATCH] package/pseudo: update version
  2016-11-26 10:50 ` Arnout Vandecappelle
  2016-11-26 14:02   ` Thomas Petazzoni
@ 2016-11-26 16:25   ` Gaël PORTAY
  1 sibling, 0 replies; 7+ messages in thread
From: Gaël PORTAY @ 2016-11-26 16:25 UTC (permalink / raw)
  To: buildroot

Hi Armout,

On Sat, Nov 26, 2016 at 11:50:45AM +0100, Arnout Vandecappelle wrote:
> 
>  Does this work? From the man page, I understand that the -S option can't be
> combined with starting a program.
> 

Yes this is working fine.

Maybe the man page should be updated :)

The -S options can be used with or without scripts.

See http://git.yoctoproject.org/cgit/cgit.cgi/pseudo/tree/pseudo.c if you want,
 * at line 337 (when pseudo is invoked without script) and
 * at line 412 (when pseudo is invoked with a script (our case)).

^ permalink raw reply	[flat|nested] 7+ messages in thread

* [Buildroot] [PATCH] package/pseudo: update version
  2016-11-05 16:34 Yann E. MORIN
@ 2016-11-05 22:34 ` Thomas Petazzoni
  0 siblings, 0 replies; 7+ messages in thread
From: Thomas Petazzoni @ 2016-11-05 22:34 UTC (permalink / raw)
  To: buildroot

Hello,

On Sat,  5 Nov 2016 17:34:54 +0100, Yann E. MORIN wrote:
> Brings in a fix about externded attributes (xattrs).
> 
> Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
> ---
>  package/pseudo/pseudo.hash | 2 +-
>  package/pseudo/pseudo.mk   | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)

Applied to master, thanks.

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

^ permalink raw reply	[flat|nested] 7+ messages in thread

* [Buildroot] [PATCH] package/pseudo: update version
@ 2016-11-05 16:34 Yann E. MORIN
  2016-11-05 22:34 ` Thomas Petazzoni
  0 siblings, 1 reply; 7+ messages in thread
From: Yann E. MORIN @ 2016-11-05 16:34 UTC (permalink / raw)
  To: buildroot

Brings in a fix about externded attributes (xattrs).

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
---
 package/pseudo/pseudo.hash | 2 +-
 package/pseudo/pseudo.mk   | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/package/pseudo/pseudo.hash b/package/pseudo/pseudo.hash
index 5822d0e..1553a29 100644
--- a/package/pseudo/pseudo.hash
+++ b/package/pseudo/pseudo.hash
@@ -1,2 +1,2 @@
 # Locally computed
-sha256  d7767c9a73b0df4f1f26952cf5eca4c6ff6fc4bc5ef15a904ea3839c70161936  pseudo-7abc9396731149df5eaf43c84fed4f3053b64de6.tar.gz
+sha256  7d4b767302f118fa1c3f89b551cf3f3f2aa92721dab86ff62f0600a394b8a81a  pseudo-45eca34c754d416a38bee90fb2d3c110a0b6cc5f.tar.gz
diff --git a/package/pseudo/pseudo.mk b/package/pseudo/pseudo.mk
index ed348f1..67538a6 100644
--- a/package/pseudo/pseudo.mk
+++ b/package/pseudo/pseudo.mk
@@ -4,7 +4,7 @@
 #
 ################################################################################
 
-PSEUDO_VERSION = 7abc9396731149df5eaf43c84fed4f3053b64de6
+PSEUDO_VERSION = 45eca34c754d416a38bee90fb2d3c110a0b6cc5f
 PSEUDO_SITE = https://git.yoctoproject.org/git/pseudo
 PSEUDO_SITE_METHOD = git
 
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2016-11-26 16:25 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-11-25 21:22 [Buildroot] [PATCH] package/pseudo: update version Gaël PORTAY
2016-11-26 10:50 ` Arnout Vandecappelle
2016-11-26 14:02   ` Thomas Petazzoni
2016-11-26 15:11     ` Arnout Vandecappelle
2016-11-26 16:25   ` Gaël PORTAY
  -- strict thread matches above, loose matches on Subject: below --
2016-11-05 16:34 Yann E. MORIN
2016-11-05 22:34 ` Thomas Petazzoni

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.