On Wed, 2021-12-08 at 22:59 +0100, Alexander Kanavin wrote:
> Signed-off-by: Alexander Kanavin <a...@linutronix.de>
> ---
>  .../0001-buckets-ssl_buckets.c-do-not-use-ERR_GET_FUNC.patch   | 3 ++-
>  ...p.creating.directories.without.sandbox-install.prefix.patch | 3 ++-
>  2 files changed, 4 insertions(+), 2 deletions(-)
> 
> diff --git 
> a/meta/recipes-support/serf/serf/0001-buckets-ssl_buckets.c-do-not-use-ERR_GET_FUNC.patch
>  
> b/meta/recipes-support/serf/serf/0001-buckets-ssl_buckets.c-do-not-use-ERR_GET_FUNC.patch
> index e6172ef5aa..5b8fb70639 100644
> --- 
> a/meta/recipes-support/serf/serf/0001-buckets-ssl_buckets.c-do-not-use-ERR_GET_FUNC.patch
> +++ 
> b/meta/recipes-support/serf/serf/0001-buckets-ssl_buckets.c-do-not-use-ERR_GET_FUNC.patch
> @@ -6,7 +6,8 @@ Subject: [PATCH] buckets/ssl_buckets.c: do not use 
> ERR_GET_FUNC
>  Upstream removed it in
>  https://github.com/openssl/openssl/pull/16004
>  
> -Upstream-Status: Pending
> +Upstream is defunct: last release in 2015, last commit in 2019.
> +Upstream-Status: Inappropriate [no upstream]
>  Signed-off-by: Alexander Kanavin <a...@linutronix.de>
>  ---
>   buckets/ssl_buckets.c | 3 +--
> diff --git 
> a/meta/recipes-support/serf/serf/SConstruct.stop.creating.directories.without.sandbox-install.prefix.patch
>  
> b/meta/recipes-support/serf/serf/SConstruct.stop.creating.directories.without.sandbox-install.prefix.patch
> index 91640d6044..cf00dbf5ac 100644
> --- 
> a/meta/recipes-support/serf/serf/SConstruct.stop.creating.directories.without.sandbox-install.prefix.patch
> +++ 
> b/meta/recipes-support/serf/serf/SConstruct.stop.creating.directories.without.sandbox-install.prefix.patch
> @@ -31,7 +31,8 @@ ERROR: scons install execution failed.
>    and the installed paths (including the paths inside libserf*.pc)
>    look correct
>  
> -Upstream-Status: Pending
> +Upstream is defunct: last release in 2015, last commit in 2019
> +Upstream-Status: Inappropriate [no upstream]
>  
>  Signed-off-by: Martin Jansa <martin.ja...@gmail.com>
>  

I'm still not convinced this is the right thing to do. I appreciate what the
docs say but I also think we may need to evolve.

Let's imagine we do have an "Inactive-Upstream" status for a minute and this
patch changes to:

Upstream-Status: Inactive-Upstream [last release 2015, last commit 2019]

If you now happen to be changing the recipe due to a new release, this is going
to trigger some kind of thought/action by seeing that status in that we might
now be able to upstream the patches. If it just says "Inappropriate", it won't
trigger the response we'd like.

We're also going to eventually end up with a whole pile of "Inappropriate"
patches, it may be useful to keep an open mind on whether any subdivision of
those would be helpful.

Does this make sense to anyone else?

Cheers,

Richard









-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#159448): 
https://lists.openembedded.org/g/openembedded-core/message/159448
Mute This Topic: https://lists.openembedded.org/mt/87599630/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to