qpid-users mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Justin Ross <justin.r...@gmail.com>
Subject Re: A change to the site code UI
Date Sun, 31 Jan 2016 15:04:32 GMT
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 <keith.wall@gmail.com> 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 <justin.ross@gmail.com> 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: users-unsubscribe@qpid.apache.org
> For additional commands, e-mail: users-help@qpid.apache.org
>
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message