<html>
<head>
<meta content="text/html; charset=windows-1252"
http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<div class="moz-cite-prefix">On 04/12/14 14:18, Gabe Alford wrote:<br>
</div>
<blockquote
cite="mid:CAGLxfGzJtfzmW2mdcWBwsE3HqvG3f-dNG4GFzxwDQBaKZ2VMQA@mail.gmail.com"
type="cite">
<div dir="ltr">
<div>Thanks for the assistance Lukas! I have an updated patch
attached.<br>
<br>
</div>
Thanks,<br>
<br>
Gabe<br>
<div>
<div>
<div class="gmail_extra"><br>
<div class="gmail_quote">On Wed, Dec 3, 2014 at 9:30 AM,
Lukas Slebodnik <span dir="ltr"><<a
moz-do-not-send="true"
href="mailto:lslebodn@redhat.com" target="_blank">lslebodn@redhat.com</a>></span>
wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0
.8ex;border-left:1px #ccc solid;padding-left:1ex">
<div>
<div>On (02/12/14 21:13), Gabe Alford wrote:<br>
>This patch removes the changelog and
Makefile.am for ipaclient as well.<br>
><br>
>Thanks,<br>
><br>
>Gabe<br>
><br>
>On Mon, Dec 1, 2014 at 8:28 AM, Martin Kosek
<<a moz-do-not-send="true"
href="mailto:mkosek@redhat.com" target="_blank">mkosek@redhat.com</a>>
wrote:<br>
><br>
>> On 12/01/2014 04:25 PM, Rob Crittenden
wrote:<br>
>> > Gabe Alford wrote:<br>
>> >><br>
>> >> On Mon, Dec 1, 2014 at 6:05 AM,
Martin Kosek <<a moz-do-not-send="true"
href="mailto:mkosek@redhat.com" target="_blank">mkosek@redhat.com</a><br>
>> >> <mailto:<a
moz-do-not-send="true"
href="mailto:mkosek@redhat.com" target="_blank">mkosek@redhat.com</a>>>
wrote:<br>
>> >><br>
>> >> On 11/30/2014 03:28 AM, Gabe
Alford wrote:<br>
>> >> > Ignore the last patch.
Updated patch attached.<br>
>> >> ><br>
>> >> > On Sat, Nov 29, 2014 at
6:03 PM, Gabe Alford<br>
>> >> <<a
moz-do-not-send="true"
href="mailto:redhatrises@gmail.com"
target="_blank">redhatrises@gmail.com</a>
<mailto:<a moz-do-not-send="true"
href="mailto:redhatrises@gmail.com"
target="_blank">redhatrises@gmail.com</a>>>
wrote:<br>
>> >> ><br>
>> >> >> This patch removes
the app_PYTHON usage.<br>
>> >> >><br>
>> >> >> Thanks,<br>
>> >> >><br>
>> >> >> Gabe<br>
>> >> >><br>
>> >> >> On Thu, Nov 27,
2014 at 9:40 AM, Martin Kosek <<a
moz-do-not-send="true"
href="mailto:mkosek@redhat.com" target="_blank">mkosek@redhat.com</a><br>
>> >> <mailto:<a
moz-do-not-send="true"
href="mailto:mkosek@redhat.com" target="_blank">mkosek@redhat.com</a>>>
wrote:<br>
>> >> >><br>
>> >> >>> Exactly, this
was the message from Martin :-) I did not test it<br>
>> >> myself,<br>
>> >> >>> but<br>
>> >> >>> removing all
app_PYTHON should be benign given we use Python<br>
>> >> setup.py<br>
>> >> >>> packaging.<br>
>> >> >>><br>
>> >> >>> On 11/27/2014
04:27 PM, Gabe Alford wrote:<br>
>> >> >>>> Thanks
guys. Sounds like it would be better to submit a
patch<br>
>> that<br>
>> >> >>> removes<br>
>> >> >>>> app_PYTHON
if it is considered dead code.<br>
>> >> >>>><br>
>> >> >>>> Gabe<br>
>> >> >>>><br>
>> >> >>>> On
Thursday, November 27, 2014, Petr Spacek <<br>
>> <a moz-do-not-send="true"
href="mailto:pspacek@redhat.com" target="_blank">pspacek@redhat.com</a><br>
>> >> <mailto:<a
moz-do-not-send="true"
href="mailto:pspacek@redhat.com" target="_blank">pspacek@redhat.com</a>>>
wrote:<br>
>> >> >>>><br>
>> >> >>>>> On
27.11.2014 11:00, Martin Basti wrote:<br>
>> >> >>>>>> On
27/11/14 00:50, Gabe Alford wrote:<br>
>> >> >>>>>>>
Hello,<br>
>> >> >>>>>>><br>
>> >>
>>>>>>> Wondering if
I could get a review. Updated patch<br>
>> >> attached.<br>
>> >> >>>>>>><br>
>> >> >>>>>>>
Thanks,<br>
>> >> >>>>>>>
Gabe<br>
>> >> >>>>>>><br>
>> >> >>>>>>>
On Tue, Nov 11, 2014 at 7:21 AM, Gabe Alford<br>
>> >> <<a
moz-do-not-send="true"
href="mailto:redhatrises@gmail.com"
target="_blank">redhatrises@gmail.com</a>
<mailto:<a moz-do-not-send="true"
href="mailto:redhatrises@gmail.com"
target="_blank">redhatrises@gmail.com</a>><br>
>> >> >>>>>
<a class="moz-txt-link-rfc2396E" href="javascript:;"><javascript:;></a><br>
>> >> >>>>>>>
<mailto:<a moz-do-not-send="true"
href="mailto:redhatrises@gmail.com"
target="_blank">redhatrises@gmail.com</a>
<mailto:<a moz-do-not-send="true"
href="mailto:redhatrises@gmail.com"
target="_blank">redhatrises@gmail.com</a><br>
>> ><br>
>> >> <a class="moz-txt-link-rfc2396E" href="javascript:;"><javascript:;></a>>>
wrote:<br>
>> >> >>>>>>><br>
>> >>
>>>>>>> Hello,<br>
>> >> >>>>>>><br>
>> >>
>>>>>>> Fix for <a
moz-do-not-send="true"
href="https://fedorahosted.org/freeipa/ticket/4700"
target="_blank">https://fedorahosted.org/freeipa/ticket/4700</a><br>
>> >> >>>>>>><br>
>> >>
>>>>>>> Thanks,<br>
>> >> >>>>>>><br>
>> >>
>>>>>>> Gabe<br>
>> >> >>>>>>><br>
>> >> >>>>>>><br>
>> >> >>>>>>><br>
>> >> >>>>>>
Hello,<br>
>> >> >>>>>><br>
>> >> >>>>>>
sorry for late response.<br>
>> >> >>>>>><br>
>> >> >>>>>> We
push this ticket to backlog, as it would be part
of build<br>
>> >> system<br>
>> >> >>>>>
refactoring.<br>
>> >> >>>>>> The
"app_PYTHON" statement is not used anymore in IPA,
the<br>
>> better<br>
>> >> >>>>>
solution is<br>
>> >> >>>>>>
remove it, instead of keeping dead code
up-to-date.<br>
>> >> >>>>><br>
>> >> >>>>> Just to
clarify:<br>
>> >> >>>>> It can
be pushed if it works, there is no need to
postpone<br>
>> >> accepting<br>
>> >> >>> patch<br>
>> >> >>>>> if<br>
>> >> >>>>> the
patch seems okay and doesn't break anything.<br>
>> >> >>>>><br>
>> >> >>>>> Martin,
please keep in mind that contributions are welcome
at<br>
>> >> any time.<br>
>> >> >>>>><br>
>> >> >>>>>
Milestones in Trac reflect our view of priorities
but it<br>
>> doesn't<br>
>> >> >>> prevent us<br>
>> >> >>>>> from
accepting correct patches from contributions at
any<br>
>> time, no<br>
>> >> >>> matter<br>
>> >> >>>>> which<br>
>> >> >>>>>
priority is stated in Trac (or even if there is no
ticket for<br>
>> >> it ...).<br>
>> >> >>>>><br>
>> >> >>>>> --<br>
>> >> >>>>> Petr^2
Spacek<br>
>> >><br>
>> >> Worked in my tests, I did
not see any breakage. I guess we can also<br>
>> >> remove the<br>
>> >>
ipa-client/ipaclient/Makefile.am while we are at
it.<br>
>> >><br>
>> >> Martin<br>
>> >><br>
>> >><br>
>> >> It looks like the
ipaclient/Makefile.am is still being used. I tried<br>
>> >> removing it and there were
errors in the build, but maybe I am wrong?<br>
>> ><br>
>> > It is needed to build ipa-join,
ipa-getkeytab and ipa-rmkeytab.<br>
>> ><br>
>> > Feel free to rip out the outdated hg
ChangeLog stuff though.<br>
>> ><br>
>> > rob<br>
>><br>
>> I think Gabe was asking about
ipa-client/ipaclient/Makefile.am and not<br>
>> about<br>
>> ipa-client/Makefile.am - we still need
this one as Rob correctly said.<br>
>><br>
>> The failure that Gabe hit in build
probably comes from the the SUBDIR<br>
>> reference<br>
>> in ipa-client/Makefile.am file. I assume
that if the reference is removed,<br>
>> the<br>
>> removal should work.<br>
>><br>
>> And yes, you can remove the Changelog too
if you are OK with it :)<br>
>><br>
>> Martin<br>
>><br>
<br>
</div>
</div>
>From d2e3176b6f6f2abb2ffbdfc198814bd1a845b876 Mon
Sep 17 00:00:00 2001<br>
>From: Gabe <<a moz-do-not-send="true"
href="mailto:redhatrises@gmail.com" target="_blank">redhatrises@gmail.com</a>><br>
>Date: Tue, 2 Dec 2014 14:43:57 -0700<br>
>Subject: [PATCH] Remove usage of app_PYTHON in
ipaserver Makefiles<br>
><br>
><a moz-do-not-send="true"
href="https://fedorahosted.org/freeipa/ticket/4700"
target="_blank">https://fedorahosted.org/freeipa/ticket/4700</a><br>
>---<br>
> ipa-client/Makefile.am | 21
---------------------<br>
> ipa-client/ipaclient/Makefile.am | 17
-----------------<br>
> ipaserver/install/Makefile.am | 27
---------------------------<br>
> ipaserver/install/plugins/Makefile.am | 24
------------------------<br>
> 4 files changed, 89 deletions(-)<br>
> delete mode 100644
ipa-client/ipaclient/Makefile.am<br>
> delete mode 100644 ipaserver/install/Makefile.am<br>
> delete mode 100644
ipaserver/install/plugins/Makefile.am<br>
><br>
>diff --git a/ipa-client/Makefile.am
b/ipa-client/Makefile.am<br>
>index
b9c7020f3b687b3c0030ed5166625e6ef07e2fa4..f6f3168774c3024e10f626b88a8952c51c0eab90
100644<br>
>--- a/ipa-client/Makefile.am<br>
>+++ b/ipa-client/Makefile.am<br>
>@@ -84,7 +84,6 @@ ipa_join_LDADD = \<br>
><br>
> SUBDIRS = \<br>
> ../asn1 \<br>
>- ipaclient \<br>
> ipa-install \<br>
> man \<br>
> $(NULL)<br>
>@@ -97,7 +96,6 @@ EXTRA_DIST = \<br>
> README \<br>
> HACKING \<br>
> NEWS \<br>
>- ChangeLog \<br>
> $(NULL)<br>
><br>
> DISTCLEANFILES = \<br>
>@@ -125,22 +123,3 @@ MAINTAINERCLEANFILES =
\<br>
> py-compile \<br>
> $(NULL)<br>
><br>
>-# Creating ChangeLog from hg log (taken from
cairo/Makefile.am):<br>
>-<br>
>-ChangeLog: $(srcdir)/ChangeLog<br>
>-<br>
>-$(srcdir)/ChangeLog:<br>
>- @if test -d "$(srcdir)/../.hg"; then \<br>
>- (cd "$(srcdir)" && \<br>
>- ./missing --run hg log --verbose) | fmt
--split-only > $@.tmp \<br>
>- && mv -f $@.tmp $@ \<br>
>- || ($(RM) $@.tmp; \<br>
>- echo Failed to generate ChangeLog,
your ChangeLog may be outdated >&2; \<br>
>- (test -f $@ || echo hg log is
required to generate this file >> $@)); \<br>
>- else \<br>
>- test -f $@ || \<br>
>- (echo A hg checkout and hg -log is
required to generate ChangeLog >&2 && \<br>
>- echo A hg checkout and hg log is required
to generate this file >> $@); \<br>
>- fi<br>
>-<br>
>-.PHONY: ChangeLog $(srcdir)/ChangeLog<br>
>diff --git a/ipa-client/ipaclient/Makefile.am
b/ipa-client/ipaclient/Makefile.am<br>
>deleted file mode 100644<br>
>index
01824b86584992fd84d4542da88395aa0e89de12..0000000000000000000000000000000000000000<br>
>--- a/ipa-client/ipaclient/Makefile.am<br>
>+++ /dev/null<br>
>@@ -1,17 +0,0 @@<br>
>-NULL =<br>
>-<br>
>-appdir = $(pythondir)/ipaclient<br>
>-app_PYTHON = \<br>
>- __init__.py \<br>
>- ipachangeconf.py \<br>
>- ipadiscovery.py \<br>
>- ntpconf.py \<br>
>- ipa_certupdate.py \<br>
>- $(NULL)<br>
>-<br>
>-EXTRA_DIST = \<br>
>- $(NULL)<br>
>-<br>
>-MAINTAINERCLEANFILES = \<br>
>- *~ \<br>
>- Makefile.in<br>
<br>
You need to remove ipa-client/ipaclient/Makefile.am
also from AC_CONFIG_FILES<br>
in file ipa-client/<a moz-do-not-send="true"
href="http://configure.ac" target="_blank">configure.ac</a><br>
<br>
<br>
It should fix problem with autoreconf.<br>
<span><font color="#888888"><br>
LS<br>
</font></span></blockquote>
</div>
</div>
</div>
</div>
</div>
</blockquote>
<br>
Sorry I can't build RPMS <br>
<br>
RPM build errors:<br>
Directory not found:
/root/freeipa/rpmbuild/BUILDROOT/freeipa-4.1.2-0.fc21.x86_64/usr/lib/python2.7/site-packages/ipaclient<br>
File not found by glob:
/root/freeipa/rpmbuild/BUILDROOT/freeipa-4.1.2-0.fc21.x86_64/usr/lib/python2.7/site-packages/ipaclient/*.py*<br>
Makefile:229: recipe for target 'rpms' failed<br>
<br>
<br>
The problem is, we don't have setup.py script for ipa client (just
for ipaserver).<br>
I suggest to remove only Changelog from ipa-client, and let other
parts of ipa-client related Makefiles untouched.<br>
<br>
Martin^2<br>
<pre class="moz-signature" cols="72">--
Martin Basti</pre>
</body>
</html>