Bug 226138 - Merge Review: metacity
Merge Review: metacity
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
All Linux
medium Severity medium
: ---
: ---
Assigned To: Parag AN(पराग)
Fedora Package Reviews List
Depends On: 203547
  Show dependency treegraph
Reported: 2007-01-31 14:40 EST by Nobody's working on this, feel free to take it
Modified: 2010-10-22 00:36 EDT (History)
4 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2010-10-22 00:36:31 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
panemade: fedora‑review+

Attachments (Terms of Use)
spec cleanup (5.38 KB, patch)
2010-10-18 04:42 EDT, Parag AN(पराग)
no flags Details | Diff

  None (edit)
Description Nobody's working on this, feel free to take it 2007-01-31 14:40:16 EST
Fedora Merge Review: metacity

Initial Owner: sandmann@redhat.com
Comment 1 Jochen Schmitt 2007-04-03 12:31:21 EDT
- Local build failed:
+ cd metacity-2.18.0
+ export LANG
+ unset DISPLAY
+ rm -rf /var/tmp/metacity-2.18.0-1-root-s4504kr
+ CPPFLAGS=' -I/var/tmp/metacity-2.18.0-1-root-s4504kr/usr/include'
+ export CPPFLAGS
+ autoreconf
src/Makefile.am: required file `./compile' not found
autoreconf: automake failed with exit status: 1
Fehler: Fehler-Status beim Beenden von /var/tmp/rpm-tmp.46901 (%build)
Comment 2 Patrice Dumas 2007-04-04 03:41:09 EDT
* a remark
the gettext_package, metacity_version and metacity_filename don't 
seem to be usefull to me, %{name}, %{version} and %{name}-%{version}
would seem more natural to me

* -I$RPM_BUILD_ROOT%{_includedir} seems wrong to me. Why is it there?

* Why run autoreconf?

* on the make line, CPPFLAGS="$CPPFLAGS" LIBS="$LIBS" seems unuseful
to me, while %{?_smp_mflags} is missing (it worked in my tests)

* The pkgconfig Requires is unuseful in main package
Comment 3 Debarshi Ray 2008-06-15 04:34:04 EDT
Comment 4 Søren Sandmann Pedersen 2008-06-16 15:13:11 EDT
I am happy enough for someone to just go ahead and fix these things. I don't
believe metacity has any ACL's.
Comment 5 Parag AN(पराग) 2010-10-18 04:42:52 EDT
Created attachment 454045 [details]
spec cleanup

Please commit this git patch that will clean this package for this review or allow to commit.
Comment 6 Parag AN(पराग) 2010-10-22 00:36:31 EDT
Committed the above patch and built in metacity-2.30.3-2.fc15


Note You need to log in before you can comment on or make changes to this bug.