lkml.org 
[lkml]   [2016]   [Sep]   [20]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH 1/8] samples: move accounting example code from Documentation
From
Date
On 09/19/2016 10:02 AM, Jonathan Corbet wrote:
> On Mon, 19 Sep 2016 08:47:32 -0600
> Shuah Khan <shuahkh@osg.samsung.com> wrote:
>
>> Move accounting examples to samples and remove it from Documentation
>> Makefile. Create a new Makefile to build accounting. It can be built
>> from top level directory or from accounting directory:
>
> So I like the basic idea of these patches; it's the direction we've been
> trying to push things for a little bit. I have a couple of specific
> comments for this one that may well apply to the others as well.
>
> 1) Is samples the right destination for these utilities? Some of them
> might well be better placed under tools/ instead. I'm not sure we've
> ever formally stated the purpose of those two subtrees; I see samples/
> as demonstrations of how to use something, while tools/ are utilities
> you might actually want to use for some purpose. Something like
> getdelays might well fall into the latter group. But maybe others
> disagree?

getdelays stumped me a bit. My first choice was tools and then I noticed
that the documentation says it is a tool and an example usage of taskstruct.
I am not set on tools or samples as location for this. I can move it to
tools instead.

>
> 2) Tools like getdelays are referenced in the documents that have been
> left behind; see accounting/delay-accounting.txt, for example. Those
> documents should be updated so that readers know where to find the
> referenced programs, wherever they end up. Converting them to Sphinx
> while you're at it is an extra-credit exercise :)

Right. Documents need updates and also the 00-Index files. I agree with
you that converting to Sphinx makes sense. There will be some learning
curve on my part. I can take this one once the move happens and do the
Documentation and 00-Index file updates.

thanks,
-- Shuah

\
 
 \ /
  Last update: 2016-09-20 18:06    [W:0.089 / U:0.044 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site