Add data-path restriction to tutorial, part 2.

Add data-path restriction to tutorial, part 2.

BUG=none
R= jf@chromium.org
TEST=none
NOTRY=true
(documentation only change)

Committed: https://crrev.com/445060e4e96898195590069cbb7b643439dc732e
Cr-Commit-Position: refs/heads/master@{#293302}

[jpmedley, 2014-09-02 20:49:31]

jmedley@chromium.org changed reviewers:
+ jfb@chromium.org

[jpmedley, 2014-09-02 20:49:31]

JF,

The CL is related to this thread:
https://groups.google.com/forum/#!msg/native-client-discuss/HBed4IvlOtI/BF4vh...

[JF, 2014-09-02 20:57:34]

https://codereview.chromium.org/531183002/diff/1/native_client_sdk/doc_genera...
File native_client_sdk/doc_generated/devguide/distributing.html (right):

https://codereview.chromium.org/531183002/diff/1/native_client_sdk/doc_genera...
native_client_sdk/doc_generated/devguide/distributing.html:148: <th
class="head"><p class="first last">Value</p>
Is this an unrelated change? Or did the html generation change and this file
wasn't updated?

https://codereview.chromium.org/531183002/diff/1/native_client_sdk/src/doc/de...
File native_client_sdk/src/doc/devguide/tutorial/tutorial-part2.rst (right):

https://codereview.chromium.org/531183002/diff/1/native_client_sdk/src/doc/de...
native_client_sdk/src/doc/devguide/tutorial/tutorial-part2.rst:306: string.
I think this is helpful, but IIUC the confusion from the original post came from
not specifying that a local web server had to be started: the user assumed file
URIs would work, but they don't.

This tutorial doesn't mention starting a server, so I think it would be good to
add this too (and keep you change here).

[jpmedley, 2014-09-03 18:57:58]

PTAL

https://codereview.chromium.org/531183002/diff/1/native_client_sdk/doc_genera...
File native_client_sdk/doc_generated/devguide/distributing.html (right):

https://codereview.chromium.org/531183002/diff/1/native_client_sdk/doc_genera...
native_client_sdk/doc_generated/devguide/distributing.html:148: <th
class="head"><p class="first last">Value</p>
I don't know. I'll remove it.

On 2014/09/02 20:57:34, JF wrote:
> Is this an unrelated change? Or did the html generation change and this file
> wasn't updated?

[JF, 2014-09-03 20:34:01]

lgtm, thanks!

[jpmedley, 2014-09-03 20:57:16]

The CQ bit was checked by jmedley@chromium.org

[commit-bot: I haz the power, 2014-09-03 20:59:41]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/jmedley@chromium.org/531183002/20001

[commit-bot: I haz the power, 2014-09-04 00:37:01]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-09-04 00:37:02]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[native-client-reviews_googlegroups.com, 2014-09-04 15:17:16]

Oops! Forgot to tell it this was documentation only. I'm on it.

Joe Medley | Technical Writer | jmedley@google.com | 816-678-7195


On Wed, Sep 3, 2014 at 6:37 PM, <commit-bot@chromium.org> wrote:

> Try jobs failed on following builders:
>   chromium_presubmit on tryserver.chromium.linux
> (http://build.chromium.org/p/tryserver.chromium.linux/
> builders/chromium_presubmit/builds/8525)
>
> https://codereview.chromium.org/531183002/
>

-- 
You received this message because you are subscribed to the Google Groups
"Native-Client-Reviews" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to native-client-reviews+unsubscribe@googlegroups.com.
To post to this group, send email to native-client-reviews@googlegroups.com.
Visit this group at http://groups.google.com/group/native-client-reviews.
For more options, visit https://groups.google.com/d/optout.

[chromium-reviews, 2014-09-04 15:17:16]

Oops! Forgot to tell it this was documentation only. I'm on it.

Joe Medley | Technical Writer | jmedley@google.com | 816-678-7195


On Wed, Sep 3, 2014 at 6:37 PM, <commit-bot@chromium.org> wrote:

> Try jobs failed on following builders:
>   chromium_presubmit on tryserver.chromium.linux
> (http://build.chromium.org/p/tryserver.chromium.linux/
> builders/chromium_presubmit/builds/8525)
>
> https://codereview.chromium.org/531183002/
>

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[jpmedley, 2014-09-04 15:19:58]

The CQ bit was checked by jmedley@chromium.org

[commit-bot: I haz the power, 2014-09-04 15:20:52]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/jmedley@chromium.org/531183002/20001

[commit-bot: I haz the power, 2014-09-04 15:23:30]

Committed patchset #2 (id:20001) as 562043c2f17d22fc075cd3375cf5ed415b51c172

[commit-bot: I haz the power, 2014-09-10 03:31:32]

Patchset 2 (id:??) landed as
https://crrev.com/445060e4e96898195590069cbb7b643439dc732e
Cr-Commit-Position: refs/heads/master@{#293302}