<div dir="ltr"> <div class="gmail_extra"><div class="gmail_quote">On Mon, Dec 1, 2014 at 6:05 AM, Martin Kosek <<a href="mailto:mkosek@redhat.com" target="_blank">mkosek@redhat.com</a>> wrote: <blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On 11/30/2014 03:28 AM, Gabe Alford wrote: > Ignore the last patch. Updated patch attached. > > On Sat, Nov 29, 2014 at 6:03 PM, Gabe Alford <<a href="mailto:redhatrises@gmail.com">redhatrises@gmail.com</a>> wrote: > >> This patch removes the app_PYTHON usage. >> >> Thanks, >> >> Gabe >> >> On Thu, Nov 27, 2014 at 9:40 AM, Martin Kosek <<a href="mailto:mkosek@redhat.com">mkosek@redhat.com</a>> wrote: >> >>> Exactly, this was the message from Martin :-) I did not test it myself, >>> but >>> removing all app_PYTHON should be benign given we use Python setup.py >>> packaging. >>> >>> On 11/27/2014 04:27 PM, Gabe Alford wrote: >>>> Thanks guys. Sounds like it would be better to submit a patch that >>> removes >>>> app_PYTHON if it is considered dead code. >>>> >>>> Gabe >>>> >>>> On Thursday, November 27, 2014, Petr Spacek <<a href="mailto:pspacek@redhat.com">pspacek@redhat.com</a>> wrote: >>>> >>>>> On 27.11.2014 11:00, Martin Basti wrote: >>>>>> On 27/11/14 00:50, Gabe Alford wrote: >>>>>>> Hello, >>>>>>> >>>>>>> Wondering if I could get a review. Updated patch attached. >>>>>>> >>>>>>> Thanks, >>>>>>> Gabe >>>>>>> >>>>>>> On Tue, Nov 11, 2014 at 7:21 AM, Gabe Alford <<a href="mailto:redhatrises@gmail.com">redhatrises@gmail.com</a> >>>>> <javascript:;> >>>>>>> <mailto:<a href="mailto:redhatrises@gmail.com">redhatrises@gmail.com</a> <javascript:;>>> wrote: >>>>>>> >>>>>>> Hello, >>>>>>> >>>>>>> Fix for <a href="https://fedorahosted.org/freeipa/ticket/4700" target="_blank">https://fedorahosted.org/freeipa/ticket/4700</a> >>>>>>> >>>>>>> Thanks, >>>>>>> >>>>>>> Gabe >>>>>>> >>>>>>> >>>>>>> >>>>>> Hello, >>>>>> >>>>>> sorry for late response. >>>>>> >>>>>> We push this ticket to backlog, as it would be part of build system >>>>> refactoring. >>>>>> The "app_PYTHON" statement is not used anymore in IPA, the better >>>>> solution is >>>>>> remove it, instead of keeping dead code up-to-date. >>>>> >>>>> Just to clarify: >>>>> It can be pushed if it works, there is no need to postpone accepting >>> patch >>>>> if >>>>> the patch seems okay and doesn't break anything. >>>>> >>>>> Martin, please keep in mind that contributions are welcome at any time. >>>>> >>>>> Milestones in Trac reflect our view of priorities but it doesn't >>> prevent us >>>>> from accepting correct patches from contributions at any time, no >>> matter >>>>> which >>>>> priority is stated in Trac (or even if there is no ticket for it ...). >>>>> >>>>> -- >>>>> Petr^2 Spacek </div></div>Worked in my tests, I did not see any breakage. I guess we can also remove the ipa-client/ipaclient/Makefile.am while we are at it. Martin </blockquote></div> </div><div class="gmail_extra">It looks like the ipaclient/Makefile.am is still being used. I tried removing it and there were errors in the build, but maybe I am wrong? </div><div class="gmail_extra">Gabe </div><div class="gmail_extra"> </div></div>