All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 1/2] less: Create symlink for pager
@ 2019-06-19  9:44 Ricardo Ribalda Delgado
  2019-06-19  9:44 ` [PATCH 2/2] dpkg: add missing running dependency Ricardo Ribalda Delgado
  2019-06-19  9:54 ` [PATCH 1/2] less: Create symlink for pager Burton, Ross
  0 siblings, 2 replies; 5+ messages in thread
From: Ricardo Ribalda Delgado @ 2019-06-19  9:44 UTC (permalink / raw)
  To: openembedded-core; +Cc: Ricardo Ribalda Delgado

Some system tools (like dpkg) expects a binary called pager when they
run on a interactive console.

This patch addes this symlink via alternative, so other packages can
also provide this symlink.

Signed-off-by: Ricardo Ribalda Delgado <ricardo@ribalda.com>
---
 meta/recipes-extended/less/less_550.bb | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/meta/recipes-extended/less/less_550.bb b/meta/recipes-extended/less/less_550.bb
index bb1618f1eb..8f59bd6087 100644
--- a/meta/recipes-extended/less/less_550.bb
+++ b/meta/recipes-extended/less/less_550.bb
@@ -24,6 +24,7 @@ LIC_FILES_CHKSUM = "file://COPYING;md5=d32239bcb673463ab874e80d47fae504 \
                     file://LICENSE;md5=ba01d0cab7f62f7f2204c7780ff6a87d \
                     "
 DEPENDS = "ncurses"
+RPROVIDES_${PN} = "pager"
 
 SRC_URI = "http://www.greenwoodsoftware.com/${BPN}/${BPN}-${PV}.tar.gz \
 	  "
@@ -39,5 +40,6 @@ do_install () {
         oe_runmake 'bindir=${D}${bindir}' 'mandir=${D}${mandir}' install
 }
 
-ALTERNATIVE_${PN} = "less"
+ALTERNATIVE_${PN} = "less pager"
 ALTERNATIVE_PRIORITY = "100"
+ALTERNATIVE_TARGET[pager] = "/usr/bin/less.less"
-- 
2.20.1



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

* [PATCH 2/2] dpkg: add missing running dependency
  2019-06-19  9:44 [PATCH 1/2] less: Create symlink for pager Ricardo Ribalda Delgado
@ 2019-06-19  9:44 ` Ricardo Ribalda Delgado
  2019-06-19  9:54 ` [PATCH 1/2] less: Create symlink for pager Burton, Ross
  1 sibling, 0 replies; 5+ messages in thread
From: Ricardo Ribalda Delgado @ 2019-06-19  9:44 UTC (permalink / raw)
  To: openembedded-core; +Cc: Ricardo Ribalda Delgado

Without this patch:

root@qt5122:~# dpkg -l
sh: pager: command not found
dpkg-query: error: showing package list on pager subprocess returned error exit status 127

Signed-off-by: Ricardo Ribalda Delgado <ricardo@ribalda.com>
---
 meta/recipes-devtools/dpkg/dpkg.inc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-devtools/dpkg/dpkg.inc b/meta/recipes-devtools/dpkg/dpkg.inc
index fee7a75814..ec9eadd21a 100644
--- a/meta/recipes-devtools/dpkg/dpkg.inc
+++ b/meta/recipes-devtools/dpkg/dpkg.inc
@@ -4,7 +4,7 @@ SECTION = "base"
 
 DEPENDS = "zlib bzip2 perl ncurses"
 DEPENDS_class-native = "bzip2-replacement-native zlib-native virtual/update-alternatives-native gettext-native perl-native"
-RDEPENDS_${PN} = "${VIRTUAL-RUNTIME_update-alternatives} perl"
+RDEPENDS_${PN} = "${VIRTUAL-RUNTIME_update-alternatives} perl pager"
 RDEPENDS_${PN}_class-native = ""
 
 UPSTREAM_CHECK_URI = "${DEBIAN_MIRROR}/main/d/dpkg/"
-- 
2.20.1



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

* Re: [PATCH 1/2] less: Create symlink for pager
  2019-06-19  9:44 [PATCH 1/2] less: Create symlink for pager Ricardo Ribalda Delgado
  2019-06-19  9:44 ` [PATCH 2/2] dpkg: add missing running dependency Ricardo Ribalda Delgado
@ 2019-06-19  9:54 ` Burton, Ross
  2019-06-19  9:57   ` Ricardo Ribalda Delgado
  2019-06-19 19:43   ` Khem Raj
  1 sibling, 2 replies; 5+ messages in thread
From: Burton, Ross @ 2019-06-19  9:54 UTC (permalink / raw)
  To: Ricardo Ribalda Delgado; +Cc: OE-core

dpkg may, but that's because dpkg is the Debian package manager and it
follows Debian traditions.  /usr/bin/pager is a Debianism: I suggest
we just patch it to use 'less'.

Ross

On Wed, 19 Jun 2019 at 10:47, Ricardo Ribalda Delgado
<ricardo@ribalda.com> wrote:
>
> Some system tools (like dpkg) expects a binary called pager when they
> run on a interactive console.
>
> This patch addes this symlink via alternative, so other packages can
> also provide this symlink.
>
> Signed-off-by: Ricardo Ribalda Delgado <ricardo@ribalda.com>
> ---
>  meta/recipes-extended/less/less_550.bb | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/meta/recipes-extended/less/less_550.bb b/meta/recipes-extended/less/less_550.bb
> index bb1618f1eb..8f59bd6087 100644
> --- a/meta/recipes-extended/less/less_550.bb
> +++ b/meta/recipes-extended/less/less_550.bb
> @@ -24,6 +24,7 @@ LIC_FILES_CHKSUM = "file://COPYING;md5=d32239bcb673463ab874e80d47fae504 \
>                      file://LICENSE;md5=ba01d0cab7f62f7f2204c7780ff6a87d \
>                      "
>  DEPENDS = "ncurses"
> +RPROVIDES_${PN} = "pager"
>
>  SRC_URI = "http://www.greenwoodsoftware.com/${BPN}/${BPN}-${PV}.tar.gz \
>           "
> @@ -39,5 +40,6 @@ do_install () {
>          oe_runmake 'bindir=${D}${bindir}' 'mandir=${D}${mandir}' install
>  }
>
> -ALTERNATIVE_${PN} = "less"
> +ALTERNATIVE_${PN} = "less pager"
>  ALTERNATIVE_PRIORITY = "100"
> +ALTERNATIVE_TARGET[pager] = "/usr/bin/less.less"
> --
> 2.20.1
>
> --
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core


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

* Re: [PATCH 1/2] less: Create symlink for pager
  2019-06-19  9:54 ` [PATCH 1/2] less: Create symlink for pager Burton, Ross
@ 2019-06-19  9:57   ` Ricardo Ribalda Delgado
  2019-06-19 19:43   ` Khem Raj
  1 sibling, 0 replies; 5+ messages in thread
From: Ricardo Ribalda Delgado @ 2019-06-19  9:57 UTC (permalink / raw)
  To: Burton, Ross; +Cc: OE-core

Hi Ross

I am ok with that.

But looking forward it is definitely more difficult to maintain that
patch on future dpkg versions. Will prepare the patch and then you
decide :P

Thanks!

On Wed, Jun 19, 2019 at 11:55 AM Burton, Ross <ross.burton@intel.com> wrote:
>
> dpkg may, but that's because dpkg is the Debian package manager and it
> follows Debian traditions.  /usr/bin/pager is a Debianism: I suggest
> we just patch it to use 'less'.
>
> Ross
>
> On Wed, 19 Jun 2019 at 10:47, Ricardo Ribalda Delgado
> <ricardo@ribalda.com> wrote:
> >
> > Some system tools (like dpkg) expects a binary called pager when they
> > run on a interactive console.
> >
> > This patch addes this symlink via alternative, so other packages can
> > also provide this symlink.
> >
> > Signed-off-by: Ricardo Ribalda Delgado <ricardo@ribalda.com>
> > ---
> >  meta/recipes-extended/less/less_550.bb | 4 +++-
> >  1 file changed, 3 insertions(+), 1 deletion(-)
> >
> > diff --git a/meta/recipes-extended/less/less_550.bb b/meta/recipes-extended/less/less_550.bb
> > index bb1618f1eb..8f59bd6087 100644
> > --- a/meta/recipes-extended/less/less_550.bb
> > +++ b/meta/recipes-extended/less/less_550.bb
> > @@ -24,6 +24,7 @@ LIC_FILES_CHKSUM = "file://COPYING;md5=d32239bcb673463ab874e80d47fae504 \
> >                      file://LICENSE;md5=ba01d0cab7f62f7f2204c7780ff6a87d \
> >                      "
> >  DEPENDS = "ncurses"
> > +RPROVIDES_${PN} = "pager"
> >
> >  SRC_URI = "http://www.greenwoodsoftware.com/${BPN}/${BPN}-${PV}.tar.gz \
> >           "
> > @@ -39,5 +40,6 @@ do_install () {
> >          oe_runmake 'bindir=${D}${bindir}' 'mandir=${D}${mandir}' install
> >  }
> >
> > -ALTERNATIVE_${PN} = "less"
> > +ALTERNATIVE_${PN} = "less pager"
> >  ALTERNATIVE_PRIORITY = "100"
> > +ALTERNATIVE_TARGET[pager] = "/usr/bin/less.less"
> > --
> > 2.20.1
> >
> > --
> > _______________________________________________
> > Openembedded-core mailing list
> > Openembedded-core@lists.openembedded.org
> > http://lists.openembedded.org/mailman/listinfo/openembedded-core
> --
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core



-- 
Ricardo Ribalda


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

* Re: [PATCH 1/2] less: Create symlink for pager
  2019-06-19  9:54 ` [PATCH 1/2] less: Create symlink for pager Burton, Ross
  2019-06-19  9:57   ` Ricardo Ribalda Delgado
@ 2019-06-19 19:43   ` Khem Raj
  1 sibling, 0 replies; 5+ messages in thread
From: Khem Raj @ 2019-06-19 19:43 UTC (permalink / raw)
  To: Burton, Ross; +Cc: Ricardo Ribalda Delgado, OE-core

[-- Attachment #1: Type: text/plain, Size: 2197 bytes --]

On Wed, Jun 19, 2019 at 2:55 AM Burton, Ross <ross.burton@intel.com> wrote:

> dpkg may, but that's because dpkg is the Debian package manager and it
> follows Debian traditions.  /usr/bin/pager is a Debianism: I suggest
> we just patch it to use 'less'.


Moreover I don’t see pager binary on my system here

>
>
> Ross
>
> On Wed, 19 Jun 2019 at 10:47, Ricardo Ribalda Delgado
> <ricardo@ribalda.com> wrote:
> >
> > Some system tools (like dpkg) expects a binary called pager when they
> > run on a interactive console.
> >
> > This patch addes this symlink via alternative, so other packages can
> > also provide this symlink.
> >
> > Signed-off-by: Ricardo Ribalda Delgado <ricardo@ribalda.com>
> > ---
> >  meta/recipes-extended/less/less_550.bb | 4 +++-
> >  1 file changed, 3 insertions(+), 1 deletion(-)
> >
> > diff --git a/meta/recipes-extended/less/less_550.bb
> b/meta/recipes-extended/less/less_550.bb
> > index bb1618f1eb..8f59bd6087 100644
> > --- a/meta/recipes-extended/less/less_550.bb
> > +++ b/meta/recipes-extended/less/less_550.bb
> > @@ -24,6 +24,7 @@ LIC_FILES_CHKSUM =
> "file://COPYING;md5=d32239bcb673463ab874e80d47fae504 \
> >                      file://LICENSE;md5=ba01d0cab7f62f7f2204c7780ff6a87d
> \
> >                      "
> >  DEPENDS = "ncurses"
> > +RPROVIDES_${PN} = "pager"
> >
> >  SRC_URI = "http://www.greenwoodsoftware.com/${BPN}/${BPN}-${PV}.tar.gz
> \
> >           "
> > @@ -39,5 +40,6 @@ do_install () {
> >          oe_runmake 'bindir=${D}${bindir}' 'mandir=${D}${mandir}' install
> >  }
> >
> > -ALTERNATIVE_${PN} = "less"
> > +ALTERNATIVE_${PN} = "less pager"
> >  ALTERNATIVE_PRIORITY = "100"
> > +ALTERNATIVE_TARGET[pager] = "/usr/bin/less.less"
> > --
> > 2.20.1
> >
> > --
> > _______________________________________________
> > Openembedded-core mailing list
> > Openembedded-core@lists.openembedded.org
> > http://lists.openembedded.org/mailman/listinfo/openembedded-core
> --
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
>

[-- Attachment #2: Type: text/html, Size: 3935 bytes --]

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

end of thread, other threads:[~2019-06-19 19:43 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-06-19  9:44 [PATCH 1/2] less: Create symlink for pager Ricardo Ribalda Delgado
2019-06-19  9:44 ` [PATCH 2/2] dpkg: add missing running dependency Ricardo Ribalda Delgado
2019-06-19  9:54 ` [PATCH 1/2] less: Create symlink for pager Burton, Ross
2019-06-19  9:57   ` Ricardo Ribalda Delgado
2019-06-19 19:43   ` Khem Raj

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.