[Freeipa-devel] [freeipa PR#188][comment] Move Python build artefacts to top level directory

2016-10-27 Thread stlaz
  URL: https://github.com/freeipa/freeipa/pull/188
Title: #188: Move Python build artefacts to top level directory

stlaz commented:
"""
+1 with @pspacek, build artefacts should be in the same directory as is their 
source. I would like to have them removed on `make clean` if that does not 
currently work.
"""

See the full comment at 
https://github.com/freeipa/freeipa/pull/188#issuecomment-256583611
-- 
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

[Freeipa-devel] [freeipa PR#188][comment] Move Python build artefacts to top level directory

2016-10-27 Thread pspacek
  URL: https://github.com/freeipa/freeipa/pull/188
Title: #188: Move Python build artefacts to top level directory

pspacek commented:
"""
Why this change? It goes against the usual behavior where build artifacts are 
placed in the same directory as sources. Also, I intend to support 
ouf-of-source-tree builds (aka 
[VPATH](https://www.gnu.org/software/automake/manual/html_node/VPATH-Builds.html))
 and I'm not sure it will work with your patch.

I would reject this change if there is no strong need.
"""

See the full comment at 
https://github.com/freeipa/freeipa/pull/188#issuecomment-256555896
-- 
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

[Freeipa-devel] [freeipa PR#188][comment] Move Python build artefacts to top level directory

2016-10-26 Thread tiran
  URL: https://github.com/freeipa/freeipa/pull/188
Title: #188: Move Python build artefacts to top level directory

tiran commented:
"""
I ran into some issues with ```make rpm```. ```setup.py``` started to pick up 
build artefacts of other packages. Now every package uses its own subdirectory 
under the top-level ```build``` directory.
"""

See the full comment at 
https://github.com/freeipa/freeipa/pull/188#issuecomment-256323114
-- 
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

[Freeipa-devel] [freeipa PR#188][comment] Move Python build artefacts to top level directory

2016-10-26 Thread tiran
  URL: https://github.com/freeipa/freeipa/pull/188
Title: #188: Move Python build artefacts to top level directory

tiran commented:
"""
The PR will also simplifies @pspacek next branch a bit.
"""

See the full comment at 
https://github.com/freeipa/freeipa/pull/188#issuecomment-256312577
-- 
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code