All of lore.kernel.org
 help / color / mirror / Atom feed
From: Alejandro del Castillo <alejandro.delcastillo@ni.com>
To: Huang Qiyu <huangqy.fnst@cn.fujitsu.com>,
	openembedded-core@lists.openembedded.org
Subject: Re: [PATCH v2] opkg: 0.3.4 -> 0.3.5
Date: Wed, 26 Jul 2017 11:26:23 -0500	[thread overview]
Message-ID: <c609be97-70a0-1717-b6ba-9a0e53420bbc@ni.com> (raw)
In-Reply-To: <1501061170-3010-1-git-send-email-huangqy.fnst@cn.fujitsu.com>

master already have opkg 0.3.5:

http://git.openembedded.org/openembedded-core/commit/meta/recipes-devtools/opkg?id=89b8cbfd65ae6c682908da8028d86692926e4d45

On 07/26/2017 04:26 AM, Huang Qiyu wrote:
> 1) Update opkg from 0.3.4 to 0.3.5.
> 2) Delete status-conffile.patch, since it is integrated upstream
> 
> Signed-off-by: Huang Qiyu <huangqy.fnst@cn.fujitsu.com>
> ---
>  .../opkg/opkg/status-conffile.patch                | 69 ----------------------
>  .../opkg/{opkg_0.3.4.bb => opkg_0.3.5.bb}          |  5 +-
>  2 files changed, 2 insertions(+), 72 deletions(-)
>  delete mode 100644 meta/recipes-devtools/opkg/opkg/status-conffile.patch
>  rename meta/recipes-devtools/opkg/{opkg_0.3.4.bb => opkg_0.3.5.bb} (93%)
> 
> diff --git a/meta/recipes-devtools/opkg/opkg/status-conffile.patch b/meta/recipes-devtools/opkg/opkg/status-conffile.patch
> deleted file mode 100644
> index 6fc405b..0000000
> --- a/meta/recipes-devtools/opkg/opkg/status-conffile.patch
> +++ /dev/null
> @@ -1,69 +0,0 @@
> -Upstream-Status: Submitted
> -Signed-off-by: Ross Burton <ross.burton@intel.com>
> -
> -From 086d5083dfe0102368cb7c8ce89b0c06b64ca773 Mon Sep 17 00:00:00 2001
> -From: Ross Burton <ross.burton@intel.com>
> -Date: Tue, 10 Jan 2017 15:24:59 +0000
> -Subject: [PATCH 1/2] opkg_cmd: only look at conffile status if we're going to
> - output it
> -
> -The loop to compare the recorded conffile hash with their hash on disk is
> -outputted at level INFO but the loop was executed at level NOTICE and higher.
> -
> -This means that if a conffile had been deleted the status operation would
> -produce error messages for output it isn't displaying.
> -
> -Signed-off-by: Ross Burton <ross.burton@intel.com>
> ----
> - libopkg/opkg_cmd.c | 2 +-
> - 1 file changed, 1 insertion(+), 1 deletion(-)
> -
> -diff --git a/libopkg/opkg_cmd.c b/libopkg/opkg_cmd.c
> -index ba57c6a..37416fd 100644
> ---- a/libopkg/opkg_cmd.c
> -+++ b/libopkg/opkg_cmd.c
> -@@ -638,7 +638,7 @@ static int opkg_info_status_cmd(int argc, char **argv, int installed_only)
> - 
> -         pkg_formatted_info(stdout, pkg);
> - 
> --        if (opkg_config->verbosity >= NOTICE) {
> -+        if (opkg_config->verbosity >= INFO) {
> -             conffile_list_elt_t *iter;
> -             for (iter = nv_pair_list_first(&pkg->conffiles); iter;
> -                  iter = nv_pair_list_next(&pkg->conffiles, iter)) {
> --- 
> -2.8.1
> -
> -From 225e30e0f9fa7cfeaa3f89e2713e5147ab371def Mon Sep 17 00:00:00 2001
> -From: Ross Burton <ross.burton@intel.com>
> -Date: Tue, 10 Jan 2017 15:28:47 +0000
> -Subject: [PATCH 2/2] conffile: gracefully handle deleted conffiles in
> - conffile_has_been_modified
> -
> -Handle conffiles that don't exist gracefully so that instead of showing an error
> -message from file_md5sum_alloc() a notice that the file has been deleted is
> -shown instead.
> -
> -Signed-off-by: Ross Burton <ross.burton@intel.com>
> ----
> - libopkg/conffile.c | 5 +++++
> - 1 file changed, 5 insertions(+)
> -
> -diff --git a/libopkg/conffile.c b/libopkg/conffile.c
> -index b2f2469..7b4b87b 100644
> ---- a/libopkg/conffile.c
> -+++ b/libopkg/conffile.c
> -@@ -51,6 +51,11 @@ int conffile_has_been_modified(conffile_t * conffile)
> -     }
> - 
> -     root_filename = root_filename_alloc(filename);
> -+    if (!file_exists(root_filename)) {
> -+        opkg_msg(INFO, "Conffile %s deleted\n", conffile->name);
> -+        free(root_filename);
> -+        return 1;
> -+    }
> - 
> -     md5sum = file_md5sum_alloc(root_filename);
> - 
> --- 
> -2.8.1
> diff --git a/meta/recipes-devtools/opkg/opkg_0.3.4.bb b/meta/recipes-devtools/opkg/opkg_0.3.5.bb
> similarity index 93%
> rename from meta/recipes-devtools/opkg/opkg_0.3.4.bb
> rename to meta/recipes-devtools/opkg/opkg_0.3.5.bb
> index a21fde1..3e511b6 100644
> --- a/meta/recipes-devtools/opkg/opkg_0.3.4.bb
> +++ b/meta/recipes-devtools/opkg/opkg_0.3.5.bb
> @@ -15,11 +15,10 @@ SRC_URI = "https://urldefense.proofpoint.com/v2/url?u=http-3A__downloads.yoctoproject.org_releases_-24&d=DwICAg&c=I_0YwoKy7z5LMTVdyO6YCiE2uzI1jjZZuIPelcSjixA&r=wNcrL2akRn6jfxhHaKavUrJB_C9JAMXtynjLd8ZzgXQ&m=BifePy5QrwZOSind-8ndN3lCCnR8yRdx4Vlx8YK5WmY&s=J_wAe7LX3BF1PYcPxUBNWuq4VmGmewb44c3BOV2NY4s&e= {BPN}/${BPN}-${PV}.tar.gz
>             file://opkg-configure.service \
>             file://opkg.conf \
>             file://0001-opkg_conf-create-opkg.lock-in-run-instead-of-var-run.patch \
> -           file://status-conffile.patch \
>  "
>  
> -SRC_URI[md5sum] = "6c52a065499056a196e0b45a27e392de"
> -SRC_URI[sha256sum] = "750b900b53b62a9b280b601a196f02da81091eda2f3478c509512aa5a1ec93be"
> +SRC_URI[md5sum] = "d202d09ea0932943071b842626cab13c"
> +SRC_URI[sha256sum] = "734bc21dea11262113fa86b928d09812618b3966f352350cf916a6ae0d343f32"
>  
>  inherit autotools pkgconfig systemd
>  
> 

-- 
Cheers,

Alejandro


      parent reply	other threads:[~2017-07-26 16:34 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-07-26  9:26 [PATCH v2] opkg: 0.3.4 -> 0.3.5 Huang Qiyu
2017-07-26  9:31 ` ✗ patchtest: failure for opkg: 0.3.4 -> 0.3.5 (rev2) Patchwork
2017-07-26 16:26 ` Alejandro del Castillo [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=c609be97-70a0-1717-b6ba-9a0e53420bbc@ni.com \
    --to=alejandro.delcastillo@ni.com \
    --cc=huangqy.fnst@cn.fujitsu.com \
    --cc=openembedded-core@lists.openembedded.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.