<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Dec 2, 2014 at 7:28 AM, Cole Robinson <span dir="ltr"><<a href="mailto:crobinso@redhat.com" target="_blank">crobinso@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 12/01/2014 08:52 PM, Jason Helfman wrote:<br>
</span><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">
On Fri, Nov 21, 2014 at 10:13 AM, Jason Helfman <<a href="mailto:jgh@freebsd.org" target="_blank">jgh@freebsd.org</a><br></span><span class="">
<mailto:<a href="mailto:jgh@freebsd.org" target="_blank">jgh@freebsd.org</a>>> wrote:<br>
<br>
    On Fri, Nov 21, 2014 at 5:09 AM, Cole Robinson <<a href="mailto:crobinso@redhat.com" target="_blank">crobinso@redhat.com</a><br></span><span class="">
    <mailto:<a href="mailto:crobinso@redhat.com" target="_blank">crobinso@redhat.com</a>>> wrote:<br>
<br>
        On 11/20/2014 11:15 PM, Jason Helfman wrote:<br>
<br>
            Hello,<br>
<br>
            I was able to get this imported for FreeBSD, but had to implement<br>
            a couple of<br>
            patches for it to work:<br>
<br></span>
            <a href="https://svnweb.freebsd.org/__ports/head/deskutils/virt-__manager/files/patch-__virtManager_config.py?view=__markup&pathrev=372972" target="_blank">https://svnweb.freebsd.org/__<u></u>ports/head/deskutils/virt-__<u></u>manager/files/patch-__<u></u>virtManager_config.py?view=__<u></u>markup&pathrev=372972</a><br>
            <<a href="https://svnweb.freebsd.org/ports/head/deskutils/virt-manager/files/patch-virtManager_config.py?view=markup&pathrev=372972" target="_blank">https://svnweb.freebsd.org/<u></u>ports/head/deskutils/virt-<u></u>manager/files/patch-<u></u>virtManager_config.py?view=<u></u>markup&pathrev=372972</a>><br>
            <a href="https://svnweb.freebsd.org/__ports/head/deskutils/virt-__manager/files/patch-__virtManager_console.py?view=__markup&pathrev=372972" target="_blank">https://svnweb.freebsd.org/__<u></u>ports/head/deskutils/virt-__<u></u>manager/files/patch-__<u></u>virtManager_console.py?view=__<u></u>markup&pathrev=372972</a><span class=""><br>
            <<a href="https://svnweb.freebsd.org/ports/head/deskutils/virt-manager/files/patch-virtManager_console.py?view=markup&pathrev=372972" target="_blank">https://svnweb.freebsd.org/<u></u>ports/head/deskutils/virt-<u></u>manager/files/patch-<u></u>virtManager_console.py?view=<u></u>markup&pathrev=372972</a>><br>
<br>
            This was taken obtained from:<br></span>
            <a href="https://build.opensuse.org/__package/view_file/openSUSE:__Factory/virt-manager/virt-__manager-Gtk-30.patch#" target="_blank">https://build.opensuse.org/__<u></u>package/view_file/openSUSE:__<u></u>Factory/virt-manager/virt-__<u></u>manager-Gtk-30.patch#</a><span class=""><br>
            <<a href="https://build.opensuse.org/package/view_file/openSUSE:Factory/virt-manager/virt-manager-Gtk-30.patch#" target="_blank">https://build.opensuse.org/<u></u>package/view_file/openSUSE:<u></u>Factory/virt-manager/virt-<u></u>manager-Gtk-30.patch#</a>><br>
<br>
            And is reference by a bug here:<br></span>
            <a href="https://bugzilla.suse.com/__show_bug.cgi?id=901869" target="_blank">https://bugzilla.suse.com/__<u></u>show_bug.cgi?id=901869</a><span class=""><br>
            <<a href="https://bugzilla.suse.com/show_bug.cgi?id=901869" target="_blank">https://bugzilla.suse.com/<u></u>show_bug.cgi?id=901869</a>><br>
<br>
            Comments and suggestions are welcome...<br>
<br>
<br>
        Thanks for the heads up. I added a comment in that suse bug asking for<br>
        the patch to be submitted upstream. But if that doesn't materialize in<br>
        a week, ping back here<br>
<br>
Any word?<br>
<br>
</span></blockquote>
<br>
Nope. And looking at the suse RPMs I see a ton of patches that they've never submitted upstream, some which are clear bug fixes... quite frustrating. I've attached their patch, can you try it locally and ensure it fixes your issues? If so I'll push it upstream<br>
<br>
Thanks,<br>
Cole<br>
<br></blockquote><div><br></div><div>Works great. Thanks!<br><br></div><div>It is too bad that their patches aren't contributed back. My advice is use their patches (if they work) and note "Obtained from SUSE" in commitlogs. When their software build process breaks with the next update, they will look at the logs and find out why. This may encourage them to contribute more to the project :)<br><br></div><div>Thanks,<br>Jason <br></div></div><br><br clear="all"><br>-- <br><div class="gmail_signature"><div dir="ltr">Jason Helfman          | FreeBSD Committer<br>jgh@FreeBSD.org     | <a href="http://people.freebsd.org/%7Ejgh" target="_blank">http://people.freebsd.org/~jgh</a>  | The Power to Serve<br></div></div>
</div></div>