https://bugzilla.redhat.com/show_bug.cgi?id=841001
Dan Mashal changed:
What|Removed |Added
Blocks|845694 |
--
You are receiving this mail because:
Y
https://bugzilla.redhat.com/show_bug.cgi?id=841001
--- Comment #41 from Fedora Update System ---
libmatewnck-1.4.0-3.fc17 has been pushed to the Fedora 17 stable repository.
--
You are receiving this mail because:
You are on the CC list for the bug.
_
https://bugzilla.redhat.com/show_bug.cgi?id=841001
Fedora Update System changed:
What|Removed |Added
Status|ON_QA |CLOSED
Resolution|---
https://bugzilla.redhat.com/show_bug.cgi?id=841001
Fedora Update System changed:
What|Removed |Added
Status|MODIFIED|ON_QA
--- Comment #39 from Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=841001
--- Comment #38 from Fedora Update System ---
libmatewnck-1.4.0-3.fc17 has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/libmatewnck-1.4.0-3.fc17
--
You are receiving this mail because:
You are on the CC lis
https://bugzilla.redhat.com/show_bug.cgi?id=841001
--- Comment #37 from Fedora Update System ---
libmatewnck-1.4.0-3.fc16 has been submitted as an update for Fedora 16.
https://admin.fedoraproject.org/updates/libmatewnck-1.4.0-3.fc16
--
You are receiving this mail because:
You are on the CC lis
https://bugzilla.redhat.com/show_bug.cgi?id=841001
Fedora Update System changed:
What|Removed |Added
Status|ASSIGNED|MODIFIED
--
You are receiving th
https://bugzilla.redhat.com/show_bug.cgi?id=841001
--- Comment #36 from Jon Ciesla ---
Git done (by process-git-requests).
--
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists
https://bugzilla.redhat.com/show_bug.cgi?id=841001
Dan Mashal changed:
What|Removed |Added
Depends On|839851 (mate-common)|
--
You are receiving this mail because:
Y
https://bugzilla.redhat.com/show_bug.cgi?id=841001
Dan Mashal changed:
What|Removed |Added
Flags||fedora-cvs?
--- Comment #35 from Dan Mashal
https://bugzilla.redhat.com/show_bug.cgi?id=841001
Rex Dieter changed:
What|Removed |Added
Flags|fedora-review? |fedora-review+
--- Comment #34 from Rex Die
https://bugzilla.redhat.com/show_bug.cgi?id=841001
--- Comment #33 from Dan Mashal ---
Updated spec and SRPM:
http://vicodan.fedorapeople.org/matespec/libmatewnck.spec
http://vicodan.fedorapeople.org/materpms/srpms/libmatewnck-1.4.0-3.fc17.src.rpm
--
You are receiving this mail because:
You ar
https://bugzilla.redhat.com/show_bug.cgi?id=841001
Rex Dieter changed:
What|Removed |Added
Status|NEW |ASSIGNED
Assignee|nob...@fedorapr
https://bugzilla.redhat.com/show_bug.cgi?id=841001
Thomas Spura changed:
What|Removed |Added
Blocks||845694
--
You are receiving this mail be
https://bugzilla.redhat.com/show_bug.cgi?id=841001
--- Comment #32 from Dan Mashal ---
Hi Leigh,
mate-common-1.4.0-5.fc16 and mate-common-1.4.0-5.fc17 are in the stable repos.
Also, mate-common-1.4.0-8.fc16 and mate-common-1.4.0-8.fc17 were submitted as
updates today.
Can you please test them
https://bugzilla.redhat.com/show_bug.cgi?id=841001
--- Comment #31 from leigh scott ---
(In reply to comment #30)
> This review is blocked and can't proceed till mate-common is in the repos or
> buildroot overrides are in place.
p.s don't remove the block as it's valid!!!
--
You are receiving
https://bugzilla.redhat.com/show_bug.cgi?id=841001
--- Comment #30 from leigh scott ---
This review is blocked and can't proceed till mate-common is in the repos or
buildroot overrides are in place.
--
You are receiving this mail because:
You are on the CC list for the bug.
https://bugzilla.redhat.com/show_bug.cgi?id=841001
leigh scott changed:
What|Removed |Added
Depends On||839851 (mate-common)
--
You are receiving
https://bugzilla.redhat.com/show_bug.cgi?id=841001
--- Comment #29 from leigh scott ---
(In reply to comment #26)
> @Leigh,
>
> I will look in to the .la files.
There's nothing to look at, if these files are in you package it will be an
instant review failure.
http://fedoraproject.org/wiki/Pa
https://bugzilla.redhat.com/show_bug.cgi?id=841001
--- Comment #28 from Wolfgang Ulbrich ---
(In reply to comment #26)
> @Wolfgang,
>
> Thanks for your helpful comments.
>
> Please review the build.log. This is already done without needing these
> flags.
>
You right in case of --enable-introsp
https://bugzilla.redhat.com/show_bug.cgi?id=841001
--- Comment #27 from Dan Mashal ---
libmatewnck 1.4.0
=
prefix: /usr
exec_prefix: /usr
libdir: /usr/lib64
bin
https://bugzilla.redhat.com/show_bug.cgi?id=841001
--- Comment #26 from Dan Mashal ---
@Wolfgang,
Thanks for your helpful comments.
Please review the build.log. This is already done without needing these flags.
@Leigh,
I will look in to the .la files.
--
You are receiving this mail because:
https://bugzilla.redhat.com/show_bug.cgi?id=841001
--- Comment #25 from Wolfgang Ulbrich ---
Hi Dan.
@ using extra configure flags --enable-startup-notification
I talk with Stefano-k (mate dev) today.
They use this flags already for their debian package.
DEB_CONFIGURE_EXTRA_FLAGS += --enable-gtk
https://bugzilla.redhat.com/show_bug.cgi?id=841001
--- Comment #24 from leigh scott ---
(In reply to comment #22)
> the provided .spec does use %configure, I *think* dan's wondering why you
> were suggesting using the following non-default configure flags:
> --enable-startup-notification --enabl
https://bugzilla.redhat.com/show_bug.cgi?id=841001
--- Comment #23 from leigh scott ---
(In reply to comment #20)
> (In reply to comment #17)
> > Updated. Please review.
> >
> > Spec URL: http://vicodan.fedorapeople.org/matespec/libmatewnck.spec
> > SRPM URL:
> > http://vicodan.fedorapeople.org/
https://bugzilla.redhat.com/show_bug.cgi?id=841001
--- Comment #22 from Rex Dieter ---
the provided .spec does use %configure, I *think* dan's wondering why you were
suggesting using the following non-default configure flags:
--enable-startup-notification --enable-gtk-doc-html
--
You are recei
https://bugzilla.redhat.com/show_bug.cgi?id=841001
--- Comment #21 from leigh scott ---
(In reply to comment #19)
> You actually don't even need the %configure flags with the proper build
> packages in the spec file.
I don't know where to start with that comment.
IT'S NEEDED BY ALL PACKAGES IN
https://bugzilla.redhat.com/show_bug.cgi?id=841001
--- Comment #20 from leigh scott ---
(In reply to comment #17)
> Updated. Please review.
>
> Spec URL: http://vicodan.fedorapeople.org/matespec/libmatewnck.spec
> SRPM URL:
> http://vicodan.fedorapeople.org/materpms/srpms/libmatewnck-1.4.0-2.fc1
https://bugzilla.redhat.com/show_bug.cgi?id=841001
--- Comment #19 from Dan Mashal ---
You actually don't even need the %configure flags with the proper build
packages in the spec file.
--
You are receiving this mail because:
You are on the CC list for the bug.
_
https://bugzilla.redhat.com/show_bug.cgi?id=841001
--- Comment #18 from leigh scott ---
(In reply to comment #16)
> sorry, my fault
>
> i meant this
> %prep
> %setup -q
> NOCONFIGURE=1 ./autogen.sh
>
> %build
> %configure --enable-startup-notification --enable-gtk-doc-html
> make %{?_smp_mfla
https://bugzilla.redhat.com/show_bug.cgi?id=841001
--- Comment #17 from Dan Mashal ---
Updated. Please review.
Spec URL: http://vicodan.fedorapeople.org/matespec/libmatewnck.spec
SRPM URL:
http://vicodan.fedorapeople.org/materpms/srpms/libmatewnck-1.4.0-2.fc17.src.rpm
Description: libmatewnck is
https://bugzilla.redhat.com/show_bug.cgi?id=841001
--- Comment #16 from Wolfgang Ulbrich ---
sorry, my fault
i meant this
%prep
%setup -q
NOCONFIGURE=1 ./autogen.sh
%build
%configure --enable-startup-notification --enable-gtk-doc-html
make %{?_smp_mflags}
Is this correct Leigh?
--
You are r
https://bugzilla.redhat.com/show_bug.cgi?id=841001
--- Comment #15 from Mario Blättermann ---
(In reply to comment #14)
> If it did configure during the autogen command the the options would be lost
> by running configure again in the %build section
Means, we could drop the autoconf options, co
https://bugzilla.redhat.com/show_bug.cgi?id=841001
--- Comment #14 from leigh scott ---
(In reply to comment #12)
> @ Mario
> this makes also no sense
>
> NOCONFIGURE=1 ./autogen.sh --enable-startup-notification
> --enable-gtk-doc-html
>
> better
>
> %prep
> %setup -q
> NOCONFIGURE=1 ./autogen
https://bugzilla.redhat.com/show_bug.cgi?id=841001
--- Comment #13 from leigh scott ---
(In reply to comment #10)
> OK, here we go:
>
> SPEC URL: http://mariobl.fedorapeople.org/Review/SPECS/libmatewnck.spec
>
> Scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=4340139
You need
https://bugzilla.redhat.com/show_bug.cgi?id=841001
--- Comment #12 from raveit65 ---
@ Mario
this makes also no sense
NOCONFIGURE=1 ./autogen.sh --enable-startup-notification --enable-gtk-doc-html
better
%prep
%setup -q
NOCONFIGURE=1 ./autogen.sh --enable-startup-notification --enable-gtk-doc-
https://bugzilla.redhat.com/show_bug.cgi?id=841001
--- Comment #11 from Mario Blättermann ---
For my changed package, we just have to readd
Requires:gtk-doc mate-common mate-corba
because none of them seems to be picked up automatically:
$ rpm -qpR ./libmatewnck-1*64.rpm
/sbin/ldconfig
/
https://bugzilla.redhat.com/show_bug.cgi?id=841001
--- Comment #10 from Mario Blättermann ---
OK, here we go:
SPEC URL: http://mariobl.fedorapeople.org/Review/SPECS/libmatewnck.spec
Scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=4340139
--
You are receiving this mail becaus
https://bugzilla.redhat.com/show_bug.cgi?id=841001
Mario Blättermann changed:
What|Removed |Added
CC||mario.blaetterm...@gmail.co
https://bugzilla.redhat.com/show_bug.cgi?id=841001
--- Comment #8 from Dan Mashal ---
thx. will look at it shortly.
--
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedora
https://bugzilla.redhat.com/show_bug.cgi?id=841001
--- Comment #7 from leigh scott ---
(In reply to comment #6)
> If mate-common-1.4.0-6.fc16 is in stable, the error will be gone.
Agreed.
--
You are receiving this mail because:
You are on the CC list for the bug.
__
https://bugzilla.redhat.com/show_bug.cgi?id=841001
--- Comment #6 from raveit65 ---
(In reply to comment #5)
> (In reply to comment #3)
>
> > f16 doesnt build (I guess due to older version of libtool):
> > http://koji.fedoraproject.org/koji/taskinfo?taskID=4328233
>
> Looking at the build and r
https://bugzilla.redhat.com/show_bug.cgi?id=841001
leigh scott changed:
What|Removed |Added
CC||leigh123li...@googlemail.co
https://bugzilla.redhat.com/show_bug.cgi?id=841001
--- Comment #4 from raveit65 ---
a few suggestions :)
1. You have double entries of BuildRequires !
2. A empty row before %changelog would be nice.
3. We need only this as BuildRequires
BuildRequires: glib2-devel
BuildRequires: gtk2-devel
B
https://bugzilla.redhat.com/show_bug.cgi?id=841001
--- Comment #3 from Dan Mashal ---
Ready for review:
SPEC:
http://vicodan.fedorapeople.org/matespec/libmatewnck.spec
SRPM:
http://vicodan.fedorapeople.org/materpms/srpms/libmatewnck-1.4.0-1.fc17.src.rpm
Description: libmatewnck is Window Navig
https://bugzilla.redhat.com/show_bug.cgi?id=841001
--- Comment #2 from Rex Dieter ---
Similar stuff needs work here as with mate-corba and
https://bugzilla.redhat.com/show_bug.cgi?id=841003#c10 , may be worth taking
the advice in comment #1
--
You are receiving this mail because:
You are on the
https://bugzilla.redhat.com/show_bug.cgi?id=841001
raveit65 changed:
What|Removed |Added
CC||chat-to...@raveit.de
--- Comment #1 from rave
https://bugzilla.redhat.com/show_bug.cgi?id=841001
Rex Dieter changed:
What|Removed |Added
CC||rdie...@math.unl.edu
Version|17
48 matches
Mail list logo