[Bug 889505] Review Request: libkqueue - Userspace implementation of the kqueue event notification mechanism

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

 



Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=889505

--- Comment #18 from Alec Leamas <leamas.alec@xxxxxxxxx> ---
(In reply to comment #17)
[cut]

> 
> > why have you forked the upstream project?
> 
> Primarily to maintain quick turnaround and minimize friction since Mark's
> current work is focused on a 2.0 release. I'll write him and ask if he'd
> like to create an official 1.0 maintenance branch with the changes I've made.

Basically, this motivation isn't enough IMHO. I suggest that you go back to
using the upstream sources. If you still need to fix config.inc you can use sed
-i to patch it in place. If there are other changes, add a patch and/or new
sources. The current situation will make maintenance hard and I would not let
it pass if I was reviewing it.

Of course, you should try to talk to upstream about merging the patches.
However, the first step is to create those patches, not to create a new
tarball.

BTW, when building for reviews rawhide is the preferred platform. Use mock, and
this is no problem. F17 is to old today.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=XnN2Lr17Uf&a=cc_unsubscribe
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review





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