[Bug 226599] Merge Review: xorg-x11-drv-i810

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=226599


Jason Tibbitts <tibbs@xxxxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|fedora-review?              |fedora-review+




--- Comment #3 from Jason Tibbitts <tibbs@xxxxxxxxxxx>  2008-12-05 16:44:02 EDT ---
Agreed.

And I hadn't noticed the change to configure; it has been a while since my
original comments.

Summarizing other stuff we chatted about on IRC:

The autotools calls before %configure aren't currently necessary and cause
build problems on rawhide, but then again they don't cause problems for the
release branches.

Even without the autotools calls, things fail to build in rawhide, but that is
due to new kernel stuff.

At this point I don't see any packaging-related that would hold up this ticket.
 It is probably a good idea to comment out the autotools calls if they aren't
necessary, but I don't see this as a blocker.

APPROVED

You might as well go ahead and do a regular CVS request to get the new package
name imported.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.

_______________________________________________
Fedora-package-review mailing list
Fedora-package-review@xxxxxxxxxx
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]