<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Aug 20, 2014 at 8:23 AM, Andreas Jaeger <span dir="ltr"><<a href="mailto:aj@suse.com" target="_blank">aj@suse.com</a>></span> wrote:<br>
<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 08/20/2014 03:12 PM, Shaun McCance wrote:<br>
> On Wed, 2014-08-20 at 12:37 +0200, Andreas Jaeger wrote:<br>
>> Shaun, it is not working in a virtual environment. To reproduce:<br>
>> * Install python virtualenv package<br>
>> * Run "virtualenv its"<br>
>> * Run "source its/bin/activate" to use the virtualenv<br>
>> * Run now in the virtualenv "pip install itstool"<br>
>><br>
>> It fails with:<br>
>> ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~<br>
>> error: could not create '/usr/share/itstool': Permission denied<br>
>><br>
>> ----------------------------------------<br>
>> Cleaning up...<br>
>> Command /tmp/its/bin/python -c "import<br>
>> setuptools;__file__='/tmp/itstools/build/itstool/setup.py';exec(compile(open(__file__).read().replace('\r\n',<br>
>> '\n'), __file__, 'exec'))" install --record<br>
>> /tmp/pip-M39Dm9-record/install-record.txt<br>
>> --single-version-externally-managed --install-headers<br>
>> /tmp/itstools/include/site/python2.7 failed with error code 1 in<br>
>> /tmp/its/build/itstool<br>
>> ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~<br>
>><br>
>> I suggest to not use '/usr/share/itstool/its' in setup.py. Changing this<br>
>> to 'itstool/its' fixed it for me.<br>
><br>
> It scares me a bit to have different versions of the same package<br>
> putting files in different places (pypi package vs upstream tarball).<br>
> But that's probably live-with-able.<br>
<br>
</div></div>If you do this correctly, it will install them the same way! The<br>
difference is installing system wide to /usr/share and installing<br>
locally in a virtual environment. datadir will be set properly if you do<br>
not hardcare /usr/share<br>
<div class=""><br>
>> But then it failed with a missing dependency on libxml2. It would be<br>
>> great if the tool could live without the dependency and use lxml<br>
>> instead, otherwise it needs to be setup using site-packages<br>
>> (--system-site-packages)<br>
><br>
> But this is a complete non-starter. itstool does rather intricate things<br>
> with XML. It's quite a bit more than parse-and-forget. To change it to<br>
> use lxml would require a complete rewrite, and I'm not even sure it can<br>
> be done. Most XML libraries don't let you shoot yourself in the foot as<br>
> easily as libxml2 does.<br>
<br>
</div>Both python-libxml2 and xml use libxml2 - just two different ways.<br>
<br>
But if rewriting is too hard, then let's not change it for now. It's not<br>
a ship-stopper.<br>
<div class=""><br>
<br>
>> We cannot move forward unless itstool can be executed in a virtual<br>
>> environment (using site packages is possible but not desired).<br>
>><br>
>> Shaun, could you take care of uploading a fixed package, please?<br>
><br>
> It's unfortunate the servers don't just have >=2.0.1. It's been out for<br>
> a while now. This isn't high on my list right now, and it's too late in<br>
> the release cycle to switch anyway. I'll get back to it after Juno.<br>
<br>
</div>Since this only affects manuals, we can do it any time, the change<br>
itself is not that huge and would not block us. If you don't have time<br>
right now, this is another issue...<br></blockquote><div><br></div><div><br></div><div>I think it's okay to press pause on this project as we hadn't planned for it in this release. </div><div><br></div><div>Plus, until the source is tagged appropriately, we don't see the benefit from the tool update. </div>
<div><br></div><div>So I'd like to continue to keep this in a holding pattern. </div><div><br></div><div>Thanks,</div><div>Anne</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im HOEnZb"><br>
Andreas<br>
--<br>
Andreas Jaeger aj@{<a href="http://suse.com" target="_blank">suse.com</a>,<a href="http://opensuse.org" target="_blank">opensuse.org</a>} Twitter: jaegerandi<br>
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany<br>
GF: Jeff Hawn,Jennifer Guild,Felix Imendörffer,HRB16746 (AG Nürnberg)<br>
GPG fingerprint = 93A3 365E CE47 B889 DF7F FED1 389A 563C C272 A126<br>
<br>
</div><div class="HOEnZb"><div class="h5">_______________________________________________<br>
Openstack-docs mailing list<br>
<a href="mailto:Openstack-docs@lists.openstack.org">Openstack-docs@lists.openstack.org</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-docs" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-docs</a><br>
</div></div></blockquote></div><br></div></div>