GNU bug report logs - #40485
gnu: Update libxfce4ui to 4.15.2.

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 40485 in the body.
You can then email your comments to 40485 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Naga Malleswari <nagamalli <at> riseup.net>
To: guix-patches <at> gnu.org, Danny Milosavljevic <dannym <at> scratchpost.org>
Subject: gnu: Update libxfce4ui to 4.15.2.
Date: Tue, 7 Apr 2020 18:25:08 +0530

[Message part 1 (text/plain, inline)]

Submission of patch for updating description for libxfce4ui

-- 
Regards
NagaMalli

[0001-gnu-Update-libxfce4ui-to-4.15.2.patch (text/x-patch, attachment)]

Message #8 received at 40485 <at> debbugs.gnu.org (full text, mbox):

From: Naga Malleswari <nagamalli <at> riseup.net>
To: 40485 <at> debbugs.gnu.org, Danny Milosavljevic <dannym <at> scratchpost.org>
Subject: Re: bug#40485: Acknowledgement (gnu: Update libxfce4ui to 4.15.2.)
Date: Tue, 7 Apr 2020 19:10:23 +0530

Hi Danny


 (inputs `(("libsm" ,libsm)
              ("libice" ,libice)
              ;; FIXME: required by libxfce4ui-1.pc, so should be
propagated,
              ;; but will lead to a conflict with gtk+.
              ("gtk+-2" ,gtk+-2)

I understand that xfce4-embed-plugin failed after libxfce4ui is updated
because of the conflict with the inputs. What is suggested to do.  

On 07/04/20 6:26 pm, GNU bug Tracking System wrote:
> Thank you for filing a new bug report with debbugs.gnu.org.
>
> This is an automatically generated reply to let you know your message
> has been received.
>
> Your message is being forwarded to the package maintainers and other
> interested parties for their attention; they will reply in due course.
>
> Your message has been sent to the package maintainer(s):
>  guix-patches <at> gnu.org
>
> If you wish to submit further information on this problem, please
> send it to 40485 <at> debbugs.gnu.org.
>
> Please do not send mail to help-debbugs <at> gnu.org unless you wish
> to report a problem with the Bug-tracking system.
>
-- 
Regards
NagaMalli

Message #11 received at 40485 <at> debbugs.gnu.org (full text, mbox):

From: Danny Milosavljevic <dannym <at> scratchpost.org>
To: Naga Malleswari <nagamalli <at> riseup.net>
Cc: 40485 <at> debbugs.gnu.org
Subject: Re: bug#40485: Acknowledgement (gnu: Update libxfce4ui to 4.15.2.)
Date: Tue, 7 Apr 2020 15:57:46 +0200

[Message part 1 (text/plain, inline)]

Build via

  ./pre-inst-env guix build -K libxfce4ui

Check /tmp/guix-build-xfce4-embed-plugin-1.6.0.drv-*/*/config.log for libxfce4ui
and see what the surrounding lines say.

If that doesn't help,

  ./pre-inst-env guix environment --pure libxfce4ui

then

  pkg-config --cflags --libs libxfce4ui-1

[Message part 2 (application/pgp-signature, inline)]

Message #14 received at 40485 <at> debbugs.gnu.org (full text, mbox):

From: Danny Milosavljevic <dannym <at> scratchpost.org>
To: Naga Malleswari <nagamalli <at> riseup.net>
Cc: 40485 <at> debbugs.gnu.org
Subject: Re: bug#40485: Acknowledgement (gnu: Update libxfce4ui to 4.15.2.)
Date: Tue, 7 Apr 2020 16:01:01 +0200

[Message part 1 (text/plain, inline)]

I meant:

Build via

  ./pre-inst-env guix build -K xfce4-embed-plugin

Check /tmp/guix-build-xfce4-embed-plugin-1.6.0.drv-*/*/config.log for libxfce4ui
and see what the surrounding lines say.

If that doesn't help,

  ./pre-inst-env guix environment --pure xfce4-embed-plugin

then

   pkg-config --cflags --libs libxfce4ui-1

[Message part 2 (application/pgp-signature, inline)]

Message #17 received at 40485 <at> debbugs.gnu.org (full text, mbox):

From: Danny Milosavljevic <dannym <at> scratchpost.org>
To: Naga Malleswari <nagamalli <at> riseup.net>
Cc: 40485 <at> debbugs.gnu.org
Subject: Re: [bug#40485] Acknowledgement (gnu: Update libxfce4ui to 4.15.2.)
Date: Tue, 7 Apr 2020 16:07:44 +0200

[Message part 1 (text/plain, inline)]

Then check the directories $PKG_CONFIG_PATH for libxfce4ui* .

In this case there's a libxfce4ui-2, which means they have changed the API
version because there was an incompatible change (previously it was
libxfce4ui-1).

So update xfce4-embed-plugin as well and hope they have adapted to the API
change (if not, we'll just get an error message again).

This will probably happen for quite a few packages, so it's gonna take some
work.

[Message part 2 (application/pgp-signature, inline)]

Message #20 received at 40485 <at> debbugs.gnu.org (full text, mbox):

From: Naga Malleswari <nagamalli <at> riseup.net>
To: Danny Milosavljevic <dannym <at> scratchpost.org>
Cc: 40485 <at> debbugs.gnu.org
Subject: Re: [bug#40485] Acknowledgement (gnu: Update libxfce4ui to 4.15.2.)
Date: Tue, 7 Apr 2020 19:47:40 +0530

I thought of the same but

xfce4-embed-plugin version is latest in guix. (1.6.0)


I got this error:

*** The required package libxfce4ui-1 was not found on your system.
*** Please install libxfce4ui-1 (atleast version 4.8.0) or adjust
*** the PKG_CONFIG_PATH environment variable if you

On 07/04/20 7:37 pm, Danny Milosavljevic wrote:
> Then check the directories $PKG_CONFIG_PATH for libxfce4ui* .
>
> In this case there's a libxfce4ui-2, which means they have changed the API
> version because there was an incompatible change (previously it was
> libxfce4ui-1).
>
> So update xfce4-embed-plugin as well and hope they have adapted to the API
> change (if not, we'll just get an error message again).
>
> This will probably happen for quite a few packages, so it's gonna take some
> work.

-- 
Regards
NagaMalli

Message #23 received at 40485 <at> debbugs.gnu.org (full text, mbox):

From: Danny Milosavljevic <dannym <at> scratchpost.org>
To: Naga Malleswari <nagamalli <at> riseup.net>
Cc: 40485 <at> debbugs.gnu.org
Subject: Re: [bug#40485] Acknowledgement (gnu: Update libxfce4ui to 4.15.2.)
Date: Tue, 7 Apr 2020 16:23:36 +0200

[Message part 1 (text/plain, inline)]

Upstream (the actual author) is https://docs.xfce.org/panel-plugins/xfce4-embed-plugin 
according to our page https://guix.gnu.org/packages/xfce4-embed-plugin-1.6.0/ .

Their latest release was in 2015, so I don't think they adapted to the new libxfce4ui.

That's too bad.

There are multiple possible ways to proceed:

(1) Delete xfce4-embed-plugin, if possible (if nobody is using it and if it has
security problems anyway etc), or
(2) Keep both packages of libxfce4ui.  Use the old package for
xfce4-embed-plugin and the new package for the other dependents.

Please make sure to use ./pre-inst-env guix lint xfce4-embed-plugin libxfce4ui
(with the old unchanged version) to find out whether there are known security
problems with it.

If not, I'd prefer (2).

[Message part 2 (application/pgp-signature, inline)]

Message #26 received at 40485 <at> debbugs.gnu.org (full text, mbox):

From: Naga Malleswari <nagamalli <at> riseup.net>
To: Danny Milosavljevic <dannym <at> scratchpost.org>
Cc: 40485 <at> debbugs.gnu.org
Subject: Re: [bug#40485] Acknowledgement (gnu: Update libxfce4ui to 4.15.2.)
Date: Wed, 8 Apr 2020 00:47:16 +0530

Hi

On 07/04/20 7:53 pm, Danny Milosavljevic wrote:
> (2) Keep both packages of libxfce4ui.  Use the old package for
> xfce4-embed-plugin and the new package for the other dependents.


(define-public libxfce4ui/latest
  (package
    (inherit libxfce4ui)
    (version "4.15.2")
    (source (origin
              (method url-fetch)
              (uri (string-append "https://archive.xfce.org/src/xfce/"
                                  name "/" (version-major+minor version) "/"
                                  name "-" version ".tar.bz2"))
              (sha256
               (base32
                "0069da27chxrgylbzcm9vhzpfnvkii2n2dz8g6jlwfcr82arkayb"))))))


I Added this and tried. always its showing 4.15.2 not found. Unable to
trace the error.

> If not, I'd prefer (2).

-- 
Regards
NagaMalli

Message #29 received at 40485 <at> debbugs.gnu.org (full text, mbox):

From: Danny Milosavljevic <dannym <at> scratchpost.org>
To: Naga Malleswari <nagamalli <at> riseup.net>
Cc: 40485 <at> debbugs.gnu.org
Subject: Re: [bug#40485] Acknowledgement (gnu: Update libxfce4ui to 4.15.2.)
Date: Tue, 7 Apr 2020 21:24:55 +0200

[Message part 1 (text/plain, inline)]

Hi Naga,

On Wed, 8 Apr 2020 00:47:16 +0530
Naga Malleswari <nagamalli <at> riseup.net> wrote:

> On 07/04/20 7:53 pm, Danny Milosavljevic wrote:
> > (2) Keep both packages of libxfce4ui.  Use the old package for
> > xfce4-embed-plugin and the new package for the other dependents.  
> 
> 
> (define-public libxfce4ui/latest
>   (package
>     (inherit libxfce4ui)
>     (version "4.15.2")
>     (source (origin
>               (method url-fetch)
>               (uri (string-append "https://archive.xfce.org/src/xfce/"
>                                   name "/" (version-major+minor version) "/"
>                                   name "-" version ".tar.bz2"))
>               (sha256
>                (base32
>                 "0069da27chxrgylbzcm9vhzpfnvkii2n2dz8g6jlwfcr82arkayb"))))))
> 
> 
> I Added this and tried. always its showing 4.15.2 not found. Unable to
> trace the error.

That approach is good is principle.

But I'd do it the other way around--otherwise you have to update all libxfce4ui
references in all the other packages (which presumably can use the newer
version just fine).

So better rename the old libxfce4ui variable to libxfce4ui-1 and then
update the reference in xfce4-embed-plugin to use libxfce4ui-1, and
then introduce a new libxfce4ui variable with a similar body as your
libxfce4ui/latest had.

That means that everyone still referring to the libxfce4ui variable will get
the new version automatically.

[Message part 2 (application/pgp-signature, inline)]

Message #32 received at 40485 <at> debbugs.gnu.org (full text, mbox):

From: Danny Milosavljevic <dannym <at> scratchpost.org>
To: Naga Malleswari <nagamalli <at> riseup.net>
Cc: 40485 <at> debbugs.gnu.org
Subject: Re: [bug#40485] Acknowledgement (gnu: Update libxfce4ui to 4.15.2.)
Date: Tue, 7 Apr 2020 21:48:03 +0200

[Message part 1 (text/plain, inline)]

Hi Naga,

On Wed, 8 Apr 2020 00:47:16 +0530
Naga Malleswari <nagamalli <at> riseup.net> wrote:

> (define-public libxfce4ui/latest
[...]
> 
> I Added this and tried. always its showing 4.15.2 not found. Unable to
> trace the error.

When processing which package?

[Message part 2 (application/pgp-signature, inline)]

Message #35 received at 40485 <at> debbugs.gnu.org (full text, mbox):

From: Naga Malleswari <nagamalli <at> riseup.net>
To: Danny Milosavljevic <dannym <at> scratchpost.org>
Cc: 40485 <at> debbugs.gnu.org
Subject: Re: [bug#40485] Acknowledgement (gnu: Update libxfce4ui to 4.15.2.)
Date: Thu, 9 Apr 2020 01:13:34 +0530

I am getting weird outputs.

I tried building two versions of help2man/latest and libuv/latest which
were build the same way and they worked perfectly fine.


I tried these

https://paste.debian.net/1139190/


CASE1:

 ./pre-inst-env guix build libxfce4ui   is building 4.14.1

./pre-inst-env guix build libxfce4ui-1           error: libxfce4ui-1: unknown package

./pre-inst-env guix build libxfce4ui <at> 4.15.2      error: libxfce4ui-4.15.2: unknown package


CasE 2: 
 ./pre-inst-env guix build libxfce4ui   is building 4.14.1

./pre-inst-env guix build libxfce4ui <at> 4.15.2  resulted  error: libxfce4ui-4.15.2: unknown package


Am i missing something here........... orocess which worked fine for
help2man and libuv is not working for this.



On 08/04/20 1:18 am, Danny Milosavljevic wrote:
> Hi Naga,
>
> On Wed, 8 Apr 2020 00:47:16 +0530
> Naga Malleswari <nagamalli <at> riseup.net> wrote:
>
>> (define-public libxfce4ui/latest
> [...]
>> I Added this and tried. always its showing 4.15.2 not found. Unable to
>> trace the error.
> When processing which package?

THis one

 ./pre-inst-env guix build xfce4ui

-- 
Regards
NagaMalli

Message #38 received at 40485 <at> debbugs.gnu.org (full text, mbox):

From: Danny Milosavljevic <dannym <at> scratchpost.org>
To: Naga Malleswari <nagamalli <at> riseup.net>
Cc: 40485 <at> debbugs.gnu.org
Subject: Re: [bug#40485] Acknowledgement (gnu: Update libxfce4ui to 4.15.2.)
Date: Thu, 9 Apr 2020 00:19:36 +0200

[Message part 1 (text/plain, inline)]

Hi Naga,

On Thu, 9 Apr 2020 01:13:34 +0530
Naga Malleswari <nagamalli <at> riseup.net> wrote:

> https://paste.debian.net/1139190/

Please, for archival purposes, post source code in the mailing list.  Pastebin
content will eventually vanish--and we do want to have some kind of public
record.

I'm gonna paste it here:

CASE 1:

(define-public libxfce4ui-1
  (package
    (name "libxfce4ui")
    (version "4.14.1")

(define-public libxfce4ui/latest
  (package
    (inherit libxfce4ui-1)
    (version "4.15.2")

CASE 2:
(define-public libxfce4ui
  (package
    (name "libxfce4ui")
    (version "4.14.1")


(define-public libxfce4ui/latest
  (package
    (inherit libxfce4ui)
    (version "4.15.2")
    (source (origin
              (method url-fetch)
              (uri (string-append "https://archive.xfce.org/src/xfce/"
                                  name "/" (version-major+minor version) "/"
                                  name "-" version ".tar.bz2"))
              (sha256
               (base32
                "0069da27chxrgylbzcm9vhzpfnvkii2n2dz8g6jlwfcr82arkayb"))))))

> CASE1:
> 
>  ./pre-inst-env guix build libxfce4ui   is building 4.14.1

I would have expected 4.15.2--are you sure that you overwrote the field SOURCE ?

> ./pre-inst-env guix build libxfce4ui-1           error: libxfce4ui-1: unknown package

"libxfce4ui-1" is not a package's name but a variable's name.  That won't work[1].

> ./pre-inst-env guix build libxfce4ui <at> 4.15.2      error: libxfce4ui-4.15.2: unknown package

Why is there a dash in the error message?

Should work just fine with "@".

> CasE 2: 
>  ./pre-inst-env guix build libxfce4ui   is building 4.14.1

Should build 4.15.2.

> ./pre-inst-env guix build libxfce4ui <at> 4.15.2  resulted  error: libxfce4ui-4.15.2: unknown package

Again the dash?  Why?

it works for me:

diff --git a/gnu/packages/xfce.scm b/gnu/packages/xfce.scm
index 0628b03207..ee2bba5ca1 100644
--- a/gnu/packages/xfce.scm
+++ b/gnu/packages/xfce.scm
@@ -163,7 +163,9 @@ Xfce Desktop Environment.")
 storage system.")
     (license lgpl2.0+)))
 
-(define-public libxfce4ui
+;; case 1
+
+(define-public libxfce4ui-1
   (package
     (name "libxfce4ui")
     (version "4.14.1")
@@ -201,6 +203,20 @@ storage system.")
 to share commonly used Xfce widgets among the Xfce applications.")
     (license lgpl2.0+)))
 
+(define-public libxfce4ui
+  (package
+    (inherit libxfce4ui-1)
+    (name "libxfce4ui") ; not necessary but I like it for no reason
+    (version "4.15.2")
+    (source (origin ; necessary block!!
+              (method url-fetch)
+              (uri (string-append "https://archive.xfce.org/src/xfce/"
+                                  name "/" (version-major+minor version) "/"
+                                  name "-" version ".tar.bz2"))
+              (sha256
+               (base32
+                "0069da27chxrgylbzcm9vhzpfnvkii2n2dz8g6jlwfcr82arkayb"))))))
+
 (define-public exo
   (package
     (name "exo")

$ ./pre-inst-env guix build libxfce4ui <at> 4.14.1
/gnu/store/fk1ziblzyslg9z7gpm9j410jfgwbilsr-libxfce4ui-4.14.1

$ ./pre-inst-env guix build libxfce4ui <at> 4.15.2
/gnu/store/cy5wyx1zrq9ahp6zl0wjpv0nd0a6hpkb-libxfce4ui-4.15.2

The argument to "guix build" is always[1] the field called NAME (and optionally
also the field called VERSION, after an "@" separator) of the package record.
The variable's name (define-public xxx) doesn't matter at this point[1].

Because the sha256 sum would have to change (among other things), make sure to
specify the field SOURCE also in the other package--otherwise you'll get the
old source.

Another reason to specify SOURCE in the other package follows:

Guile, the programming language used for Guix packages, usually uses
lexical scope.
Even the facility we built in order to make records (like <PACKAGE>) also uses
lexical scope.

That means a reference will bind to the identifier that is closest in the source
code (before it).  Once the identifier is bound, that's it--it's not going to
change binding ever again.

That means if you do this (silly example to prove a point--but you can actually
run that for real!):

  (define-public a
    (package
      (name "a")
      (version name)
      ;; Please ignore the following line for understanding for now.
      (source #f) (build-system #f) (synopsis #f) (description #f) (license #f) (home-page #f)))

  (define-public b
    (package
      (inherit a)
      (name "b")))

  (write (package-name b))
  (newline)
  (write (package-version b))
  (newline)

package "b" would have: (name "b") but (version "a").
Version is "a".  That's not a typo.

However, with

  (define-public a
    (package
      (name "a")
      (version name) ; here, NAME is a reference, which will be bound to the NAME one line above
      ;; Please ignore the following line for understanding for now.
      (source #f) (build-system #f) (synopsis #f) (description #f) (license #f) (home-page #f)))

  (define-public b
    (package
      (inherit a)
      (name "b")
      (version name)))

  (write (package-name b))
  (newline)
  (write (package-version b))
  (newline)

package "b" would have: (name "b") and (version "b").

Similarly for all other references, most notably the references to NAME and VERSION that
usually occur in a SOURCE field.

[1] Except for "guix build -e ..." (don't use it).
    Usually, user-facing tools only care about the contents of the <PACKAGE> record.
    Guile variables are invisible to the end user of the "guix" tool.

[Message part 2 (application/pgp-signature, inline)]

Message #41 received at 40485 <at> debbugs.gnu.org (full text, mbox):

From: Danny Milosavljevic <dannym <at> scratchpost.org>
To: Naga Malleswari <nagamalli <at> riseup.net>
Cc: 40485 <at> debbugs.gnu.org
Subject: Re: [bug#40485] Acknowledgement (gnu: Update libxfce4ui to 4.15.2.)
Date: Thu, 9 Apr 2020 00:31:13 +0200

[Message part 1 (text/plain, inline)]

Correction:

That means a reference will bind to the value of the variable with that name
that is closest in the source code (before it).

Once the reference is bound, that's it--it's not going to change binding ever
again.

[Message part 2 (application/pgp-signature, inline)]

Message #44 received at 40485 <at> debbugs.gnu.org (full text, mbox):

From: Danny Milosavljevic <dannym <at> scratchpost.org>
To: Naga Malleswari <nagamalli <at> riseup.net>
Cc: 40485 <at> debbugs.gnu.org
Subject: Re: [bug#40485] Acknowledgement (gnu: Update libxfce4ui to 4.15.2.)
Date: Thu, 9 Apr 2020 00:44:07 +0200

[Message part 1 (text/plain, inline)]

Or maybe a simpler, but less applicable, example:

(let ((x 2))
  (let ((y x))
    (let ((x 3))
      y)))

=> 2

NOT 3.

[Message part 2 (application/pgp-signature, inline)]

Message #47 received at 40485 <at> debbugs.gnu.org (full text, mbox):

From: Naga Malleswari <nagamalli <at> riseup.net>
To: Danny Milosavljevic <dannym <at> scratchpost.org>
Cc: 40485 <at> debbugs.gnu.org
Subject: Re: [bug#40485] Acknowledgement (gnu: Update libxfce4ui to 4.15.2.)
Date: Fri, 10 Apr 2020 02:12:42 +0530

[Message part 1 (text/plain, inline)]

Hi

Finally It worked......Submitting the patch for review.

Now xfce4-embed-plugin is building fine after i modified the input to
old version of libxfce4ui.

I referred the older commit message from help2man from the log. I am
still not sure about it.

Its the same code i have sent you. I used the origin block. And (name  
x) was the only difference.

On 09/04/20 3:49 am, Danny Milosavljevic wrote:
> + (inherit libxfce4ui-1)
> +    (name "libxfce4ui") ; not necessary but I like it for no reason
> +    (version "4.15.2")
> +    (source (origin ; necessary block!!
I used the same. I added name block. It worked !!!
> $ ./pre-inst-env guix build libxfce4ui <at> 4.14.1
> /gnu/store/fk1ziblzyslg9z7gpm9j410jfgwbilsr-libxfce4ui-4.14.1
>
> $ ./pre-inst-env guix build libxfce4ui <at> 4.15.2
> /gnu/store/cy5wyx1zrq9ahp6zl0wjpv0nd0a6hpkb-libxfce4ui-4.15.2

It worked the same.

1) If name block is removed in latest.   Its building the older version.
Why??

2)  /latest is also removed from variable. Would i make a difference?


> ewline)
>   (write (package-version b))
>   (newline)
Thanks for such a long explanation. It took a bit of time for me.
> package "b" would have: (name "b") and (version "b").
>
>     Guile variables are invisible to the end user of the "guix" tool.

-- 
Regards
NagaMalli

[0001-gnu-libxfce4ui-Add-4.15.2.patch (text/x-patch, attachment)]

Message #50 received at 40485 <at> debbugs.gnu.org (full text, mbox):

From: Danny Milosavljevic <dannym <at> scratchpost.org>
To: Naga Malleswari <nagamalli <at> riseup.net>, <ludo <at> gnu.org>
Cc: 40485 <at> debbugs.gnu.org
Subject: Re: [bug#40485] Acknowledgement (gnu: Update libxfce4ui to 4.15.2.)
Date: Fri, 10 Apr 2020 00:11:57 +0200

[Message part 1 (text/plain, inline)]

Hi,

On Fri, 10 Apr 2020 02:12:42 +0530
Naga Malleswari <nagamalli <at> riseup.net> wrote:

>On 09/04/20 3:49 am, Danny Milosavljevic wrote:
>> + (inherit libxfce4ui-1)
>> +    (name "libxfce4ui") ; not necessary but I like it for no reason
>> +    (version "4.15.2")
>> +    (source (origin ; necessary block!!  

>I used the same. I added name block. It worked !!!

>1) If name block is removed in the latest [libxfce4ui package], it's building the
>older version. Why??

If I remove (name "libxfce4ui") from the latest libxfce4ui package, and then
invoke

  make -j5 SUBDIRS=

then I get an error

  error: name: unbound variable.

from the compilation of gnu/packages/xfce.scm.
So that's why the latest libxfce4ui package would never be loaded into guix
(it's broken) and instead the other one would be used.

I'm not sure why that is the case.  It's certainly different to most other
programming languages.  @Ludo?

@Ludo: Reduced test case:

  ,use (guix packages)
  (define-public a
    (package
      (name "a")
      (version name)
      (source #f) (build-system #f) (synopsis #f) (description #f) (license #f) (home-page #f)))

  (define-public b
    (package
      (inherit a)
      (version name))) ; error message here

Error message:

  ;;; <stdin>:9:4: warning: possibly unbound variable `name'
  <unnamed port>:9:4: In procedure module-lookup: Unbound variable: name

What's happening here?  I would have expected NAME to be available from the
descendant record and accessing the ancestor record... is it not available
on purpose?

>2)  /latest is also removed from variable. Would it make a difference?

No.  It's just part of a variable's name (the slash has no special meaning
either).

[Message part 2 (application/pgp-signature, inline)]

Message #53 received at 40485 <at> debbugs.gnu.org (full text, mbox):

From: Naga Malleswari <nagamalli <at> riseup.net>
To: Danny Milosavljevic <dannym <at> scratchpost.org>, ludo <at> gnu.org
Cc: 40485 <at> debbugs.gnu.org
Subject: Re: [bug#40485] Acknowledgement (gnu: Update libxfce4ui to 4.15.2.)
Date: Mon, 13 Apr 2020 02:39:07 +0530

Hi Danny

Is there a suggestion how to work on this.


Meanwhile i submitted another patch, please review

http://debbugs.gnu.org/cgi/bugreport.cgi?bug=40546


On 10/04/20 3:41 am, Danny Milosavljevic wrote:
> Hi,
>
> On Fri, 10 Apr 2020 02:12:42 +0530
> Naga Malleswari <nagamalli <at> riseup.net> wrote:
>
>> On 09/04/20 3:49 am, Danny Milosavljevic wrote:
>>> + (inherit libxfce4ui-1)
>>> +    (name "libxfce4ui") ; not necessary but I like it for no reason
>>> +    (version "4.15.2")
>>> +    (source (origin ; necessary block!!  
>> I used the same. I added name block. It worked !!!
>> 1) If name block is removed in the latest [libxfce4ui package], it's building the
>> older version. Why??
> If I remove (name "libxfce4ui") from the latest libxfce4ui package, and then
> invoke
>
>   make -j5 SUBDIRS=
>
> then I get an error
>
>   error: name: unbound variable.
>
> from the compilation of gnu/packages/xfce.scm.
> So that's why the latest libxfce4ui package would never be loaded into guix
> (it's broken) and instead the other one would be used.
>
> I'm not sure why that is the case.  It's certainly different to most other
> programming languages.  @Ludo?
>
> @Ludo: Reduced test case:
>
>   ,use (guix packages)
>   (define-public a
>     (package
>       (name "a")
>       (version name)
>       (source #f) (build-system #f) (synopsis #f) (description #f) (license #f) (home-page #f)))
>
>   (define-public b
>     (package
>       (inherit a)
>       (version name))) ; error message here
>
> Error message:
>
>   ;;; <stdin>:9:4: warning: possibly unbound variable `name'
>   <unnamed port>:9:4: In procedure module-lookup: Unbound variable: name
>
> What's happening here?  I would have expected NAME to be available from the
> descendant record and accessing the ancestor record... is it not available
> on purpose?
>
>> 2)  /latest is also removed from variable. Would it make a difference?
> No.  It's just part of a variable's name (the slash has no special meaning
> either).

-- 
Regards
NagaMalli

Message #56 received at 40485 <at> debbugs.gnu.org (full text, mbox):

From: Danny Milosavljevic <dannym <at> scratchpost.org>
To: Naga Malleswari <nagamalli <at> riseup.net>
Cc: 40485 <at> debbugs.gnu.org
Subject: Re: [bug#40485] Acknowledgement (gnu: Update libxfce4ui to 4.15.2.)
Date: Mon, 13 Apr 2020 21:27:17 +0200

[Message part 1 (text/plain, inline)]

Hi Naga,

when trying to build the dependents, I get a build failure in
xfce4-equake-plugin.

Please fix xfce4-equake-plugin to also use libxfce4ui-1.
Same for xfce4-kbdleds-plugin.
Same for xfce4-mailwatch-plugin.

Otherwise OK.

[Message part 2 (application/pgp-signature, inline)]

Message #59 received at 40485 <at> debbugs.gnu.org (full text, mbox):

From: Naga Malleswari <nagamalli <at> riseup.net>
To: Danny Milosavljevic <dannym <at> scratchpost.org>
Cc: 40485 <at> debbugs.gnu.org
Subject: [PATCH v3] : [bug#40485] Acknowledgement (gnu: Update libxfce4ui to
 4.15.2.)
Date: Tue, 14 Apr 2020 21:13:05 +0530

[Message part 1 (text/plain, inline)]

Hi


Submitting v3 of patch

On 14/04/20 12:57 am, Danny Milosavljevic wrote:
> Hi Naga,
>
> when trying to build the dependents, I get a build failure in
> xfce4-equake-plugin.
My mistake i checked only embed plugin and formatted the patch.
>
> Please fix xfce4-equake-plugin to also use libxfce4ui-1.
> Same for xfce4-kbdleds-plugin.
> Same for xfce4-mailwatch-plugin.
Yes i build without modifications and after modifying each to validate
and i made necessary changes.
>
> Otherwise OK.

-- 
Regards
NagaMalli

[0001-gnu-libxfce4ui-Add-4.15.2.patch (text/x-patch, attachment)]

Message #62 received at 40485 <at> debbugs.gnu.org (full text, mbox):

From: Tobias Geerinckx-Rice <me <at> tobias.gr>
To: Danny Milosavljevic <dannym <at> scratchpost.org>, 40485 <at> debbugs.gnu.org
Subject: Re: [bug#40485] Acknowledgement (gnu: Update libxfce4ui to 4.15.2.)
Date: Tue, 14 Apr 2020 19:07:33 +0200

[Message part 1 (text/plain, inline)]

Naga, Danny,

What's the reason for adding an unstable version?  We don't 
usually do that in Guix.

Is this required for some other package?

Sorry for not bringing this up sooner; I'm still slogging through 
the backlog in a most random way.

Kind regards,

T G-R

[signature.asc (application/pgp-signature, inline)]

Message #67 received at 40485-close <at> debbugs.gnu.org (full text, mbox):

From: Danny Milosavljevic <dannym <at> scratchpost.org>
To: Tobias Geerinckx-Rice <me <at> tobias.gr>, Naga Malleswari
 <nagamalli <at> riseup.net>
Cc: 40485-close <at> debbugs.gnu.org
Subject: Re: [bug#40485] Acknowledgement (gnu: Update libxfce4ui to 4.15.2.)
Date: Sun, 19 Apr 2020 17:59:28 +0200

[Message part 1 (text/plain, inline)]

Hi T G-R,
Hi Naga,

whoops, I've forgotten that that would be an unstable version.

Thanks for telling us.

Let's not merge this update then.

I've already merged the update of libxfce4util to 4.15.0--should we revert that?

Naga, please if possible let's only add stable releases to Guix.
If they are experimental, let's not add them to Guix if it can be helped.

[Message part 2 (application/pgp-signature, inline)]

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.