[Bug 1170875] Review Request: budgie-desktop - An elegant desktop with GNOME integration

2020-02-19 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1170875

Michel Alexandre Salim  changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution|--- |NOTABUG
Last Closed||2020-02-19 17:47:07



--- Comment #35 from Michel Alexandre Salim  ---
No longer working on this, freeing it up so someone else can get this reviewed

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1170875] Review Request: budgie-desktop - An elegant desktop with GNOME integration

2017-08-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1170875



--- Comment #34 from Yamin  ---
Will we get it on Fedora 26?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1170875] Review Request: budgie-desktop - An elegant desktop with GNOME integration

2016-09-12 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1170875



--- Comment #33 from haevalen...@gmail.com ---
Fantastic, Budgie Desktop has improved a lot lately and it would be fantastic
availability in fedora or as an additional spin.

However, it has some flaws in usability as the inconsistency of some of its
components (for being in development), such as the Network applet or absence of
a switch graphic applications (Alt + Tab).

Is there a way to actually prove it? I mean it can be downloaded here:
https://software.opensuse.org/download.html?project=home%3Aikeydoherty%3Asolus-project&package=budgie-desktop
but it requires manual installation of some components of GNOME, which is not
very friendly.

I hope you can have this desktop fedora 26 or 27, but the volunteer time is
valuable. Thank you.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1170875] Review Request: budgie-desktop - An elegant desktop with GNOME integration

2016-02-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1170875



--- Comment #32 from mikey  ---
My initial plan was to get this running in a reasonable packaged form on my
Chromebook. But it would be nice to get it packaged up in a state that it could
be included for the community.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1170875] Review Request: budgie-desktop - An elegant desktop with GNOME integration

2016-02-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1170875

Zbigniew Jędrzejewski-Szmek  changed:

   What|Removed |Added

 CC||mic...@michel-slm.name
  Flags||needinfo?(michel@michel-slm
   ||.name)



--- Comment #31 from Zbigniew Jędrzejewski-Szmek  ---
Michel, what are your plans with this?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1170875] Review Request: budgie-desktop - An elegant desktop with GNOME integration

2016-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1170875



--- Comment #30 from Rex Dieter  ---
Oh, and for OnlyShowIn=... I'd recommend patching things to use/recognize
X-Budgie instead for now, at least until Budgie is registered formally.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1170875] Review Request: budgie-desktop - An elegant desktop with GNOME integration

2016-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1170875



--- Comment #29 from Rex Dieter  ---
Re: .desktop validation in comment #26 ...

You really only need to validate stuff under /usr/share/applications (ie, stuff
intended to be displayed in application menus)

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1170875] Review Request: budgie-desktop - An elegant desktop with GNOME integration

2016-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1170875



--- Comment #28 from mikey  ---
I haven't tried to remove "libgnome-volume-control" from my updated spec file
as I'm not sure how to go about it, but perhaps with some guidance we can get
it packaged up in an acceptable form.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1170875] Review Request: budgie-desktop - An elegant desktop with GNOME integration

2016-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1170875



--- Comment #27 from mikey  ---
Created attachment 1121946
  --> https://bugzilla.redhat.com/attachment.cgi?id=1121946&action=edit
Updated patch for budgie desktop 10.2.2

Here is the updated patch to go along with the spec file.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1170875] Review Request: budgie-desktop - An elegant desktop with GNOME integration

2016-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1170875



--- Comment #26 from mikey  ---
Created attachment 1121945
  --> https://bugzilla.redhat.com/attachment.cgi?id=1121945&action=edit
Updated SPEC for budgie desktop 10.2.2

This is my attempt to update the original spec to budgie desktop 10.2.2. 

I had some issues with the validation of the desktop files so had to disable
it. The reason it was failing was for the "xsession" desktop file there is an
entry "DesktopNames" for ligthDM, which is not in the XDG specification,
looking briefly into it they seemed to be saying that this file is not a "real"
desktop file so this entry shouldn't be added to the specification. Also the
other desktop files failed as they had a 'OnlyShowIn=Budgie' which the
validator does not recognise.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1170875] Review Request: budgie-desktop - An elegant desktop with GNOME integration

2016-02-02 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1170875

mikey  changed:

   What|Removed |Added

 CC||abc.mi...@googlemail.com



--- Comment #25 from mikey  ---
Just wondering what the current state of play is with this?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1170875] Review Request: budgie-desktop - An elegant desktop with GNOME integration

2015-10-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1170875



--- Comment #24 from Upstream Release Monitoring 
 ---
pbrobinson's scratch build of
gnome-dvb-daemon?#451a118009234eac9e3f06f465b00cf93478a5d5 for
git://pkgs.fedoraproject.org/gnome-dvb-daemon?#451a118009234eac9e3f06f465b00cf93478a5d5
and rawhide failed http://koji.fedoraproject.org/koji/taskinfo?taskID=11560895

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1170875] Review Request: budgie-desktop - An elegant desktop with GNOME integration

2015-10-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1170875



--- Comment #23 from Zbigniew Jędrzejewski-Szmek  ---
In light of the recent FESCo decision, the bundling would be allowed. Would be
nice to make another attempt at this.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1170875] Review Request: budgie-desktop - An elegant desktop with GNOME integration

2015-08-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1170875

Neal Gompa  changed:

   What|Removed |Added

 CC||ngomp...@gmail.com



--- Comment #22 from Neal Gompa  ---
I'm looking forward to seeing this in F23 myself, too.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1170875] Review Request: budgie-desktop - An elegant desktop with GNOME integration

2015-05-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1170875



--- Comment #21 from Manilal  ---
It would be really nice if we could include Budgie in Fedora 23. It's really a
good desktop with sleek UI and high performance.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1170875] Review Request: budgie-desktop - An elegant desktop with GNOME integration

2015-02-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1170875



--- Comment #20 from Michel Alexandre Salim  ---
Will try and find the time to do this in the next couple of days - I don't have
access to Fedora at work.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1170875] Review Request: budgie-desktop - An elegant desktop with GNOME integration

2015-02-19 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1170875



--- Comment #19 from Zbigniew Jędrzejewski-Szmek  ---
Ping?

Build for new rawhide:
http://koji.fedoraproject.org/koji/taskinfo?taskID=816.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1170875] Review Request: budgie-desktop - An elegant desktop with GNOME integration

2015-01-27 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1170875



--- Comment #18 from Zbigniew Jędrzejewski-Szmek  ---
So, I think the FPC decision is suboptimal, but let's try to get this done. Can
you check if you can build budgie with
http://koji.fedoraproject.org/koji/taskinfo?taskID=8746278? If yes, I'll submit
a review request.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1170875] Review Request: budgie-desktop - An elegant desktop with GNOME integration

2014-12-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1170875



--- Comment #17 from Michel Alexandre Salim  ---
https://fedorahosted.org/fpc/ticket/476#comment:6

The copylib exception is rejected, we'd have to turn libgnome-volume-control
into a shared lib. I'll follow up on the ticket and dev + packaging lists
asking who should undertake that and whether the review should wait till then.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1170875] Review Request: budgie-desktop - An elegant desktop with GNOME integration

2014-12-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1170875



--- Comment #16 from Zbigniew Jędrzejewski-Szmek  ---
(In reply to Michel Alexandre Salim from comment #15)
> Meanwhile, let's continue the review on the assumption that an exemption
> will be granted (the other packages that uses g-v-c will have to properly
> declare that they bundle g-v-c -- currently not being done).
OK.

> I'll release a -3 update with the appdata file and a
> bundled(gnome-volume-control) virtual provide. One question though -- what
> version should this virtual provide use?
Date is used in the examples I checked.

bundled(gnome-volume-control) = 20130904

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1170875] Review Request: budgie-desktop - An elegant desktop with GNOME integration

2014-12-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1170875



--- Comment #15 from Michel Alexandre Salim  ---
I just opened a request to determine whether libgnome-volume-control is an
appropriate copylib with FPC: https://fedorahosted.org/fpc/ticket/476

Meanwhile, let's continue the review on the assumption that an exemption will
be granted (the other packages that uses g-v-c will have to properly declare
that they bundle g-v-c -- currently not being done).

I'll release a -3 update with the appdata file and a
bundled(gnome-volume-control) virtual provide. One question though -- what
version should this virtual provide use?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1170875] Review Request: budgie-desktop - An elegant desktop with GNOME integration

2014-12-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1170875



--- Comment #14 from Zbigniew Jędrzejewski-Szmek  ---
Ah, FPC not FESCo. Sorry, there's a link to FESCo in the next paragraph which I
copied by mistake.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1170875] Review Request: budgie-desktop - An elegant desktop with GNOME integration

2014-12-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1170875

Rex Dieter  changed:

   What|Removed |Added

 CC||rdie...@math.unl.edu



--- Comment #13 from Rex Dieter  ---
See here
http://fedoraproject.org/wiki/Packaging:No_Bundled_Libraries#Exceptions

For details about bundled library policy, and how to ask for exceptions, in
particular.  (hint: comment #12 is incorrect)

that said, the exception here should be trivial (this code aparrently is
intended to be bundled as is used in those other referenced gnome components)

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1170875] Review Request: budgie-desktop - An elegant desktop with GNOME integration

2014-12-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1170875



--- Comment #12 from Zbigniew Jędrzejewski-Szmek  ---
My last comment is wrong. Even if it is a copy library, FESCo is supposed to
give its stamp of approval. Please file a ticket for a bundling exception at
https://fedorahosted.org/fesco/newticket.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1170875] Review Request: budgie-desktop - An elegant desktop with GNOME integration

2014-12-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1170875



--- Comment #11 from Zbigniew Jędrzejewski-Szmek  ---
Ah, OK, that's why I couldn't find it. According to [1] the submodule does not
have a stable api and is supposed to be "linked" wherever it is needed. So
including it in budgie-desktop is allowed by the packaging guidelines, as a
"copy library".

[1] https://mail.gnome.org/archives/gnomecc-list/2012-October/msg3.html

Neverthless, all four projects use the same git version of the submodule. It
seems a bit unfortunate to have it compiled separately four times. 
It would be great to de-duplicate those copies at some point, but it's nothing
to hold up the review over.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1170875] Review Request: budgie-desktop - An elegant desktop with GNOME integration

2014-12-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1170875



--- Comment #10 from Michel Alexandre Salim  ---
(In reply to Zbigniew Jędrzejewski-Szmek from comment #7)
> budgie bundles https://git.gnome.org/browse/libgnome-volume-control. If I
> read things correctly, it does not use the system version. It probably
> should.

There's no system version -- gnome-control-center, gnome-settings-daemon and
gnome-shell all bundle libgnome-volume-control using git submodules:

https://mail.gnome.org/archives/commits-list/2012-November/msg06793.html
https://bugzilla.gnome.org/show_bug.cgi?id=686488

(just checked and indeed there is no standalone libgnome-volume-control lib). I
could check with the desktop and packaging lists on this one just to make sure.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1170875] Review Request: budgie-desktop - An elegant desktop with GNOME integration

2014-12-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1170875



--- Comment #9 from Michel Alexandre Salim  ---
Updated spec and SRPM:
https://salimma.fedorapeople.org/specs/desktop/budgie-desktop.spec
https://salimma.fedorapeople.org/specs/desktop/budgie-desktop-8-2.fc21.src.rpm


(In reply to Zbigniew Jędrzejewski-Szmek from comment #7)
> desktop-file-install should be used. See
> http://fedoraproject.org/wiki/Packaging:Guidelines#desktop-file-
> install_usage.
> 
Done

> Package should own /usr/share/gir-1.0, /usr/share/vala/vapi, /usr/share/vala.
> 
Done

Let me work on the AppData file, and check on the other two items with
upstream.
> .appdata.xml file should be added. See

> 
> > %description devel
> > Development files for the Budgie Desktop
> I'd like to see a more verbose explanation here. bugie-desktop-devel is
> needed to develop plugins? Please clarify that.
> 
> budgie bundles https://git.gnome.org/browse/libgnome-volume-control. If I
> read things correctly, it does not use the system version. It probably
> should.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1170875] Review Request: budgie-desktop - An elegant desktop with GNOME integration

2014-12-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1170875

lnxslck  changed:

   What|Removed |Added

 CC|bquei...@gmail.com  |



--- Comment #8 from Michel Alexandre Salim  ---
(In reply to Ralf Corsepius from comment #6)
> (In reply to Michel Alexandre Salim from comment #5)
> 
> > On that note, what's the difference between ${prefix} and %prefix% ? Thanks.
> 
> Let me try to elaborate.
> 
> When building, the *.spec receives values/settings from rpm. 
> Inside of the spec, these rpm-provided values are referenced as %{} and
> used to propagate them into a package's build process. 
> Here, to "configure", as options being passed to configure. configure
> interprets/processes these parameters and generates source-files from "*.in"
> files, substituting all "@name@" patterns with the corresponding values.
> 


Ah, whereas the $-prefixed variables are only local to the *.pc itself, and
configure doesn't touch them at all! Thanks. I somehow missed the line that
hard-coded prefix to /usr and assumed they're two forms of expansions.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1170875] Review Request: budgie-desktop - An elegant desktop with GNOME integration

2014-12-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1170875



--- Comment #7 from Zbigniew Jędrzejewski-Szmek  ---
desktop-file-install should be used. See
http://fedoraproject.org/wiki/Packaging:Guidelines#desktop-file-install_usage.

.appdata.xml file should be added. See
http://people.freedesktop.org/~hughsient/appdata/ for some introduction.
Screenshots on 
https://evolve-os.com/budgie/ are probably OK, so this should only be a matter
of writing the description.

> %description devel
> Development files for the Budgie Desktop
I'd like to see a more verbose explanation here. bugie-desktop-devel is needed
to develop plugins? Please clarify that.

Package should own /usr/share/gir-1.0, /usr/share/vala/vapi, /usr/share/vala.

budgie bundles https://git.gnome.org/browse/libgnome-volume-control. If I read
things correctly, it does not use the system version. It probably should.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1170875] Review Request: budgie-desktop - An elegant desktop with GNOME integration

2014-12-05 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1170875



--- Comment #6 from Ralf Corsepius  ---
(In reply to Michel Alexandre Salim from comment #5)

> On that note, what's the difference between ${prefix} and %prefix% ? Thanks.

Let me try to elaborate.

When building, the *.spec receives values/settings from rpm. 
Inside of the spec, these rpm-provided values are referenced as %{} and
used to propagate them into a package's build process. 
Here, to "configure", as options being passed to configure. configure
interprets/processes these parameters and generates source-files from "*.in"
files, substituting all "@name@" patterns with the corresponding values.

In this case, this means

The spec's 
%configure
expands to
./configure .. --libdir=%{_libdir} ...
expands to 
./configure .. --libdir=/usr/lib64 ...

./configure interprets the values being passed as argument to --libdir=...
and substitutes the sed pattern @libdir@ in all *.ins with the value it
received through --libdir=...

In this case, configure creates a file budgie-plugin/budgie-1.0.pc
using budgie-plugin/budgie-1.0.pc.in as input files, substituing 
@libdir@ with /usr/lib64

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1170875] Review Request: budgie-desktop - An elegant desktop with GNOME integration

2014-12-05 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1170875



--- Comment #5 from Michel Alexandre Salim  ---
(In reply to Ralf Corsepius from comment #3)
> Some remarks:
> 

> +prefix=@prefix@
> +exec_prefix=@prefix@
> +libdir=@libdir@
> +includedir=@includedir@
> +datarootdir=@datarootdir@
> 
On that note, what's the difference between ${prefix} and %prefix% ? Thanks.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1170875] Review Request: budgie-desktop - An elegant desktop with GNOME integration

2014-12-05 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1170875



--- Comment #4 from Michel Alexandre Salim  ---
(In reply to Ralf Corsepius from comment #3)
> Some remarks:
> 
> - The "# Adjust for Fedora's multilib directory layout"
> section from %build is a hack playing with the symtops of an upstream bug
> (They miss to propagate some configuration parameters to "*.pc").
> 
> I'd propose this patch instead:
> 

Right. Was thinking of patching the .pc.in file directly but wasn't sure if
that layout (with pkgconfig hard-coded to /usr/lib but the libraries themselves
elsewhere) might make sense in some other distribution.

But we should at least try and upstream it, so I'll submit a pull request with
the patch and, meanwhile, use it instead of dynamically using sed

> 
> - Building is non-verbose.
> You should append --disable-silent-rules to %configure.

Fixing it in -2, thanks.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1170875] Review Request: budgie-desktop - An elegant desktop with GNOME integration

2014-12-05 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1170875

Ralf Corsepius  changed:

   What|Removed |Added

 CC||rc040...@freenet.de



--- Comment #3 from Ralf Corsepius  ---
Some remarks:

- The "# Adjust for Fedora's multilib directory layout"
section from %build is a hack playing with the symtops of an upstream bug (They
miss to propagate some configuration parameters to "*.pc").

I'd propose this patch instead:

--- budgie-plugin/budgie-1.0.pc.in
+++ budgie-plugin/budgie-1.0.pc.in
@@ -1,8 +1,8 @@
-prefix=/usr
-exec_prefix=${prefix}
-libdir=${exec_prefix}/lib
-includedir=${prefix}/include
-datarootdir=${prefix}/share
+prefix=@prefix@
+exec_prefix=@prefix@
+libdir=@libdir@
+includedir=@includedir@
+datarootdir=@datarootdir@

 Name: Budgie
 Description: Budgie Library


- Building is non-verbose.
You should append --disable-silent-rules to %configure.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1170875] Review Request: budgie-desktop - An elegant desktop with GNOME integration

2014-12-05 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1170875

Zbigniew Jędrzejewski-Szmek  changed:

   What|Removed |Added

 CC||zbys...@in.waw.pl
   Assignee|nob...@fedoraproject.org|zbys...@in.waw.pl



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1170875] Review Request: budgie-desktop - An elegant desktop with GNOME integration

2014-12-05 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1170875



--- Comment #2 from Michel Alexandre Salim  ---
(In reply to lnxslck from comment #1)
> I second that. Budgie is a clean and good looking DE, a good alternative for
> those who don't like Gnome or have a computer with less horse power.

Who prefer not to run Gnome Shell, that is. Budgie actually integrates really
quite well with the Gnome stack, and indeed the umbrella Evolve OS is a "Friend
of Gnome" (see its homepage).

Would appreciate if you could review this (if you can't for some reason, that's
fine too, of course)

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1170875] Review Request: budgie-desktop - An elegant desktop with GNOME integration

2014-12-05 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1170875

lnxslck  changed:

   What|Removed |Added

 CC||bquei...@gmail.com



--- Comment #1 from lnxslck  ---
I second that. Budgie is a clean and good looking DE, a good alternative for
those who don't like Gnome or have a computer with less horse power.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review