xen-devel.lists.xenproject.org archive mirror
 help / color / mirror / Atom feed
From: Lars Kurth <lars.kurth@citrix.com>
To: Ian Jackson <Ian.Jackson@citrix.com>
Cc: Wei Liu <wei.liu2@citrix.com>,
	Lars Kurth <lars.kurth.xen@gmail.com>,
	Andrew Cooper <Andrew.Cooper3@citrix.com>,
	George Dunlap <dunlapg@umich.edu>,
	David Vrabel <david.vrabel@citrix.com>,
	xen-devel <xen-devel@lists.xenproject.org>,
	Boris Ostrovsky <boris.ostrovsky@oracle.com>,
	Anshul Makkar <anshul.makkar@citrix.com>,
	Roger Pau Monne <roger.pau@citrix.com>
Subject: Re: xenbits "official" repo for XTF (was Re: [PATCH 0/2] xtf: add launcher (+1 bugfix)
Date: Mon, 18 Jul 2016 15:36:29 +0000	[thread overview]
Message-ID: <D3B2B4DD.2B639%lars.kurth@citrix.com> (raw)
In-Reply-To: <22412.62984.651830.877889@mariner.uk.xensource.com>



On 18/07/2016 16:30, "Ian Jackson" <ian.jackson@eu.citrix.com> wrote:

>Lars Kurth writes ("Re: [Xen-devel] xenbits "official" repo for XTF (was
>Re: [PATCH 0/2] xtf: add launcher (+1 bugfix)"):
>> I don't object to a new xtf directory in the docs root per se. I mainly
>> object to how the documents are presented in
>> http://xenbits.xen.org/docs/unstable
>
>The documents under there are generated from xen.git#master.
>
>> As far as I understand, the docs directory structure maps onto headline
>> categories in http://xenbits.xen.org/docs/unstable. Is this correct?
>
>What ?  No.
>
>docs/ is just the HTTP namespace section for documents hosted on
>xenbits.  Its subdirectories currently include:
>
>  - one directory for each existing stable and unstable Xen branch,
>    generated from the in-tree documentation source.
>
>  - some example osstest output, referred to from the osstest.git
>    in-tree README
>
>  - a subdirectory `xcp' which is presumably now obsolete and retained
>    for historical reasons.
>
>It appears that it doesn't have an index.html, so we get a webserver
>directory listing.
>
>> I would only object to creating a new docs/xtf directory,
>> - iff directory structure is exposed in the web index in
>> http://xenbits.xen.org/docs/unstable are linked, and
>
>I don't understand what you are asking.
>
>> - iff there is only one document in this directory now, and
>> - iff this will likely be the case in future.
>
>I'm proposing that the initial contents of the xtf/ directory would be
>the initial contents of the /people/andrewcoop/xen-test-framework/
>directory, only with the titles changed as discussed.
>
>So there would be one `document', which is really an HTML page, called
>index.html, so it would replace the webserver directory listing.

>
>> And I would have the same concern for osstest. If there are several
>> documents already in each, then go ahead and create them.
>
>Is your concern that it is a bad idea to create a directory rather
>than a file ?
>
>I think this is wrongheaded.
>
>It is better to create a directory now, with an index.html.  Otherwise
>it will become difficult to turn it into a directory later.

In that case, forget my objection and go ahead and create the directories.
I clearly don't understand how the docs toolchain works and was assuming
that the docs directory structure gets exposed in the web server index,
without being able to specify an extra indirection. Given that is not the
case, the directory structure does not matter.

Lars

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel

  reply	other threads:[~2016-07-18 15:36 UTC|newest]

Thread overview: 43+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-05-19 13:46 [PATCH 0/2] xtf: add launcher (+1 bugfix) Roger Pau Monne
2016-05-19 13:46 ` [PATCH 1/2] xtf: remove setting ROOT path in common.mk Roger Pau Monne
2016-05-19 13:52   ` Andrew Cooper
2016-05-19 13:46 ` [PATCH 2/2] xtf: add a launcher script Roger Pau Monne
2016-06-09 15:04 ` xenbits "official" repo for XTF (was Re: [PATCH 0/2] xtf: add launcher (+1 bugfix) Ian Jackson
2016-06-13  8:59   ` George Dunlap
2016-06-13 10:10   ` Andrew Cooper
2016-06-13 14:03     ` George Dunlap
2016-06-13 14:11       ` Ian Jackson
2016-06-14 17:27         ` Andrew Cooper
2016-06-15  8:31           ` George Dunlap
2016-06-20 17:03             ` Ian Jackson
2016-06-20 17:10               ` Konrad Rzeszutek Wilk
2016-06-21 10:17               ` Wei Liu
2016-06-21 10:36               ` David Vrabel
2016-06-21 10:42                 ` Ian Jackson
2016-07-01 19:04                 ` Ian Jackson
2016-07-06 11:19                   ` Lars Kurth
2016-07-06 14:22                     ` Konrad Rzeszutek Wilk
2016-07-07 11:10                       ` Lars Kurth
2016-07-07 11:26                         ` Andrew Cooper
2016-07-07 13:59                           ` Ian Jackson
2016-07-07 14:02                             ` David Vrabel
2016-07-07 14:58                               ` Ian Jackson
2016-07-07 15:00                                 ` Ian Jackson
2016-07-07 14:11                             ` Andrew Cooper
2016-07-07 17:17                               ` Lars Kurth
2016-07-08 13:06                                 ` Andrew Cooper
2016-07-08 13:42                                   ` Lars Kurth
2016-07-08 13:48                                     ` Andrew Cooper
2016-07-12 15:13                                   ` George Dunlap
2016-07-15 10:21                                 ` Ian Jackson
2016-07-15 10:49                                   ` Lars Kurth
2016-07-18 14:36                                     ` Ian Jackson
2016-07-18 15:19                                       ` Lars Kurth
2016-07-18 15:30                                         ` Ian Jackson
2016-07-18 15:36                                           ` Lars Kurth [this message]
2016-07-18 15:58                                             ` Ian Jackson
2016-07-18 16:06                                               ` Lars Kurth
2016-07-18 15:31                                         ` Ian Jackson
2016-08-08 17:04                                   ` Ian Jackson
2016-07-07 15:22                           ` Konrad Rzeszutek Wilk
2016-06-14 17:01   ` Konrad Rzeszutek Wilk

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=D3B2B4DD.2B639%lars.kurth@citrix.com \
    --to=lars.kurth@citrix.com \
    --cc=Andrew.Cooper3@citrix.com \
    --cc=Ian.Jackson@citrix.com \
    --cc=anshul.makkar@citrix.com \
    --cc=boris.ostrovsky@oracle.com \
    --cc=david.vrabel@citrix.com \
    --cc=dunlapg@umich.edu \
    --cc=lars.kurth.xen@gmail.com \
    --cc=roger.pau@citrix.com \
    --cc=wei.liu2@citrix.com \
    --cc=xen-devel@lists.xenproject.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).