vpdupdate[22016]: src/helper_functions.cpp:103: The command "/usr/sbin/iprconfig" is not executable.

[bugproxy]

Problem Description
==================================
This error
vpdupdate[22016]: src/helper_functions.cpp:103: The command "/usr/sbin/iprconfig" is not executable.
keeps showing up in syslog

it appears that vpdupdate has wrong path for iprconfig

# uname -a
Linux gdl1mabp1 4.4.0-36-generic #55-Ubuntu SMP Thu Aug 11 18:00:57 UTC 2016 ppc64le ppc64le ppc64le GNU/Linux

== Comment: #5 - VASANT HEGDE <email address hidden> - 2016-12-09 00:33:12 ==
I think below upstream commit will solve this issue.

commit 17f4c3c86be356f7d5914dfc79337d3cd1eabe50
Author: Ankit Kumar <email address hidden>
Date: Thu Dec 8 14:57:28 2016 +0530

    Add function to search command in standard path

    Different distros install commands in different location. One of the example
    is iprconfig. Redhat and suse installs it under /usr/sbin but ubuntu places
    this binary inside /sbin. Current lsvpd code always searches iprconfig binary
    inside /usr/sbin as this path is hard-coded and call gets failed in case
    binary is not present in that particular location.

[Michael Hohnbaum (hohnbaum)]

Steve,

Can you please add this to the queue for Foundations. Thanks.

                     Michael

On 02/20/2017 10:19 PM, Launchpad Bug Tracker wrote:
> bugproxy (bugproxy) has assigned this bug to you for Ubuntu:
>
> Problem Description
> ==================================
> This error
> vpdupdate[22016]: src/helper_functions.cpp:103: The command "/usr/sbin/iprconfig" is not executable.
> keeps showing up in syslog
>
> it appears that vpdupdate has wrong path for iprconfig
>
> # uname -a
> Linux gdl1mabp1 4.4.0-36-generic #55-Ubuntu SMP Thu Aug 11 18:00:57 UTC 2016 ppc64le ppc64le ppc64le GNU/Linux
>
> == Comment: #5 - VASANT HEGDE <email address hidden> - 2016-12-09 00:33:12 ==
> I think below upstream commit will solve this issue.
>
> commit 17f4c3c86be356f7d5914dfc79337d3cd1eabe50
> Author: Ankit Kumar <email address hidden>
> Date: Thu Dec 8 14:57:28 2016 +0530
>
> Add function to search command in standard path
>
> Different distros install commands in different location. One of the example
> is iprconfig. Redhat and suse installs it under /usr/sbin but ubuntu places
> this binary inside /sbin. Current lsvpd code always searches iprconfig binary
> inside /usr/sbin as this path is hard-coded and call gets failed in case
> binary is not present in that particular location.
>
> ** Affects: ubuntu
> Importance: Undecided
> Assignee: Taco Screen team (taco-screen-team)
> Status: New
>
>
> ** Tags: architecture-ppc64le bugnameltc-145767 severity-medium targetmilestone-inin16041

--
Michael Hohnbaum
OIL Program Manager
Power (ppc64el) Development Project Manager
Canonical, Ltd.

[Steve Langasek (vorlon)]

Where is the "upstream" for this package? The only source listed in debian/copyright of the package is http://sourceforge.net/projects/linux-diag/files/libvpd/, which points to https://git.code.sf.net/p/linux-diag/libvpd linux-diag-libvpd for the git repository. The listed commit is not present in that repository.

[bugproxy (bugproxy)]

------- Comment From <email address hidden> 2017-03-21 23:43 EDT-------
(In reply to comment #14)
> Where is the "upstream" for this package? The only source listed in
> debian/copyright of the package is
> http://sourceforge.net/projects/linux-diag/files/libvpd/, which points to
> https://git.code.sf.net/p/linux-diag/libvpd linux-diag-libvpd for the git
> repository. The listed commit is not present in that repository.

git clone https://git.code.sf.net/p/linux-diag/lsvpd linux-diag-lsvpd

-Vasant

[Launchpad Janitor (janitor)]

This bug was fixed in the package lsvpd - 1.7.7-1ubuntu2

---------------
lsvpd (1.7.7-1ubuntu2) zesty; urgency=medium

  * debian/patches/Add-function-to-search-command-in-standard-path.patch:
    Cherry-pick upstream patch to support finding iprconfig on the path
    instead of in a fixed location. LP: #1666412.

 -- Steve Langasek <email address hidden> Wed, 22 Mar 2017 08:33:32 -0700