<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Mar 21, 2014 at 8:36 AM, Doug Hellmann <span dir="ltr"><<a href="mailto:doug.hellmann@dreamhost.com" target="_blank">doug.hellmann@dreamhost.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 dir="ltr"><div style="font-size:small">There is quite a list of un-released changes to hacking:</div>

<div style="font-size:small"><br></div><div><div>
* Make H202 check honor pep8 #noqa comment</div><div>* Updated from global requirements</div><div>* Updated from global requirements</div><div>* Switch over to oslosphinx</div>
<div>* HACKING.rst: Fix odd indentation in an example code</div><div>* Remove tox locale overrides</div><div>* Updated from global requirements</div><div>
* Clarify H403 message</div><div>* More portable way to detect modules for H302</div><div>* Fix python 3 incompatibility in _get_import_type</div><div>* Trigger warnings for raw and unicode docstrings</div>
<div>* Enhance H233 rule</div><div>* Add check for removed modules in Python 3</div><div>* Add Python3 deprecated assert* to HACKING.rst</div><div>
* Turn Python3 section into a list</div><div>* Re-Add section on assertRaises(Exception</div><div>* Cleanup HACKING.rst</div><div>* Move hacking guide to root directory</div>
<div>* Fix typo in package summary</div><div>* Add H904: don't wrap lines using a backslash</div><div>* checking for metaclass to be Python 3.x compatible</div>
<div>* Remove unnecessary headers</div><div>* Add -U to pip install command in tox.ini</div><div>* Fix typos of comment in module core</div><div>
* Updated from global requirements</div><div>* Add a check for file with only comments</div><div>* Enforce grouping like imports together</div><div>* Add noqa support for H201 (bare except)</div>
<div>* Enforce import grouping</div><div>* Clean up how test env variables are parsed</div><div>* Fix the escape character</div><div>
* Remove vim modeline sample</div><div>* Add a check for newline after docstring summary</div></div><div class="gmail_extra"><br></div><div class="gmail_extra"><div style="font-size:small">
It looks like it might be time for a new release anyway, especially if it resolves the packaging issue you describe.</div></div></div></blockquote><div><br></div><div><br></div><div>I think two new releases are needed. I have been holding off cutting the next hacking release until we are closer to Juno. Since the next release will include new rules I didn't want to distract anyone from focusing on stabilizing Icehouse.</div>

<div><br></div><div>So it sounds like we need:</div><div><br></div><div>* Hacking 0.8.1 to fix the oslo.sphinx  oslosphinx issue for Icehouse. Since we cap hacking versions at 0.9 [1] this will  get used in icehouse.</div>

<div>* Hacking 0.9 to release all the new hacking goodness. This will be targeted for use in Juno.</div><div><br></div><div>[1] <a href="https://review.openstack.org/#/c/81356/">https://review.openstack.org/#/c/81356/</a><br>

</div><div> </div><div><br></div><div>If this sounds good, I will cut 0.8.1 this afternoon.</div><div><br></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 dir="ltr"><div class="gmail_extra"><div style="font-size:small"><br></div><div style="font-size:small">
As far as the symlink, I think that's a potentially bad idea. It's only going to encourage the continued use of oslo.sphinx. Since the package is only needed to build the documentation, and not to actually use the tool, I don't think we need the symlink in place, do we?</div>

<span class=""><font color="#888888">
<div style="font-size:small"><br></div><div style="font-size:small">Doug</div></font></span><div><div class="h5"><br><br><div class="gmail_quote">On Fri, Mar 21, 2014 at 6:17 AM, Thomas Goirand <span dir="ltr"><<a href="mailto:zigo@debian.org" target="_blank">zigo@debian.org</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">Hi,<br>
<br>
The current version of python-hacking wants python-oslo.sphinx, but<br>
we're moving to python-oslosphinx. In Debian, I made python-oslo.sphinx<br>
as a transition empty package that only depends on python-oslosphinx. As<br>
a consequence, python-hacking needs to be updated to use<br>
python-oslosphinx, otherwise it wont have available build-dependencies.<br>
<br></blockquote></div></div></div></div></div></blockquote><div><br></div><div><br></div><div>Thank you for bringing this to our attention, I wonder how we can detect this in our CI system in the future to prevent this.</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 dir="ltr"><div class="gmail_extra"><div>
<div class="h5">
<div class="gmail_quote"><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">
I was also thinking about providing a symlink from oslo/sphinx to<br>
oslosphinx. Maybe it'd be nice to have this directly in oslosphinx?<br>
<br>
Thoughts anyone?<br>
<br>
Cheers,<br>
<br>
Thomas<br>
<br>
_______________________________________________<br>
OpenStack-dev mailing list<br>
<a href="mailto:OpenStack-dev@lists.openstack.org" target="_blank">OpenStack-dev@lists.openstack.org</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</blockquote></div><br></div></div></div></div>
</blockquote></div><br></div></div>