<div dir="ltr">Munging together some replies:<br><div class="gmail_extra"><br><br><div class="gmail_quote">On Sat, Jun 29, 2013 at 12:26 PM, Monty Taylor <span dir="ltr"><<a href="mailto:mordred@inaugust.com" target="_blank">mordred@inaugust.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div class="im">On 06/29/2013 02:14 PM, Thomas Goirand wrote:<br>

> On 06/30/2013 12:16 AM, Mark Washenberger wrote:<br>
>> Hi Thomas,<br>
>><br>
>> We switched to using entrypoints for creating our binaries.<br>
><br>
> Can you explain this a bit more?<br></div></blockquote><div><br></div><div style>Very specifically, <a href="https://github.com/openstack/glance/blob/master/setup.py#L55">https://github.com/openstack/glance/blob/master/setup.py#L55</a></div>
<div style><br></div><div style>This means rather than keeping a /bin directory in our project, we're asking setup.py to generate scripts and install them in an appropriate location when setup.py install is run.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div class="im">
><br>
>> Unfortunately, this means you need to have the glance version<br>
>> "installed" in some sense in your path when you want to run those tests.<br>
>> This felt like a no-no when we made the change, but it worked fine<br>
>> without any changes to our normal tox and run_tests.sh (with virtualenv)<br>
>> so it made it through. Please let me know if this is really hard to deal<br>
>> with on your side; I can imagine a few other possible fixes we might pursue.<br>
>><br>
>> Thanks<br>
><br>
> Using virtualenv is indeed not an option on the packaging side. Having<br>
> glance build-depends: on itself is also not an option.<br>
><br>
> What can I do?<br></div></blockquote><div><br></div><div style>Just for future reference for me, what kinds of dependencies can you establish for your "test" step in maintaining your package repos? Do you pretty much only have access to a source checkout / tarball? And is the "test" step completely contained within the "build" step?</div>
<div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div class="im">
<br>
</div>I believe we should fix the unittests to not shell out to glance-manage<br>
in that manner. One of the nice things about moving the code from bin/<br>
to glance.cmd is that it's available inside of the source tree for<br>
unittests! :) What we want to do is call the glance.cmd.manage:main()<br>
function directly. So, for instance, in:<br>
<br>
glance/tests/functional/__init__.py<br>
and<br>
glance/tests/functional/test_glance_manage.py<br>
<br>
We can call the function itself, rather than the shell wrapper.<br>
<br>
I'll make a patch. Are you seeing problems with anything other than<br>
glance-manage?<br></blockquote><div><br></div><div style>Thanks Monty! This was exactly what I had in mind as an alternative approach. I believe glance-manage is the only binary where glance had such an install dependency for testing--but I can't answer for Thomas.</div>
<div><br></div><div> </div></div></div></div>