Hi, Keith. Thanks for finding that failure. I've committed a fix.
http://svn.apache.org/viewvc/qpid/site/python/generate.py?r1=1727820&r2=1727819&pathrev=1727820 On the other question, I think that may be fair behavior for the gen-* scripts. In this specific instance, it's the removes in gen_books that are doing this: http://svn.apache.org/repos/asf/qpid/site/scripts/gen-java-release-books If we didn't do that, we'd be mingling the output of multiple docbook invocations. My feeling is that it's better to resolve that manually when the output changes, rather than overlay multiple outputs. It seems like there's a very good chance that changes to a shared file would lead to inconsistent output. This problem shouldn't arise very often. We only rarely change the structure of the docbook output. Justin On Sun, Jan 31, 2016 at 1:20 AM, Keith W <[email protected]> wrote: > Hi Justin, > > I gave the change a test drive. It looks like a useful improvement. > > I did notice a couple of problems when trying to render trunk, which I > don't think are related to the change. > > 1) Running "make clean gen-cpp-release RELEASE=trunk" dies with the > following message. > > gen-cpp-release-api-doc: Removing > 'input/releases/qpid-cpp-trunk/messaging-api' > Traceback (most recent call last): > File "scripts/gen-cpp-release-api-doc", line 88, in <module> > gen_messaging_api(release, source_dir, release_dir) > File "scripts/gen-cpp-release-api-doc", line 27, in gen_messaging_api > gen_cpp_messaging_api(release, source_dir, component_dir) > File "scripts/gen-cpp-release-api-doc", line 45, in gen_cpp_messaging_api > output_dir=output_dir) > File "/home/keith/src/site/python/generate.py", line 196, in gen_doxygen > append(path, local_conf) > NameError: global name 'local_conf' is not defined > > > 2) "make clean gen-java-release RELEASE=trunk" works successfully, but > existing svn owned directories beneath input/ get removed by script, > which upsets the SVN. > I think this problem will occur when republishing any release that > already exists in the tree. I see: > > keith@ubuntu:~/src/site$ svn status > ~ input/releases/qpid-java-trunk/jms-client-0-8 > ~ input/releases/qpid-java-trunk/java-broker > M input/releases/qpid-java-trunk/index.md > > The same problem happens with gen-cpp-release RELEASE=trunk. > > What do you think? > > cheers, Keith > > > On 29 January 2016 at 01:01, Justin Ross <[email protected]> wrote: > > I made a usability change that impacts those who update the Qpid site > code. > > > > Previously "make render" would render once to the output/ dir (so you > could > > look at with your browser) and once to the docs/ dir (with its URLs ready > > for publication). > > > > Now I've isolated the publish step. Use "make render" as before to > update > > output/ and see the rendered content. Once you've iterated and you're > > satisfied with the changes, run "make publish". It renders to docs/ and > > finishes by printing the svn status of your docs/ dir and reminding you > > that there's one more thing to do: > > > > https://gist.github.com/ssorj/063434d2c2acb44b8aae > > > > The upside is that "make render", which one typically does more > frequently, > > is a little faster, and "make publish" does a little more to help you see > > what you need to commit. > > > > Justin > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [email protected] > For additional commands, e-mail: [email protected] > >
