On Fri, 18 Aug 2017, Rodrigo Vivi <[email protected]> wrote:
> On my own workflow I was missing a way to download mboxes
> directly from patchwork with the patchwork id. So my first
> reflex was to modify dim to fulfil my needs. However that
> was increasing dim in complexity and dependencies and leaving
> that messy.
>
> That was when Jani suggested me the dimrc extension with the
> example that is now part of this spec.
>
> That was clean and simple enough to understand, so Daniel
> suggested me to add it to the spec.
>
> For record let's put my final local solution that lays now on
> my own ~/.dimrc
>
> dim_pwaq()
> {
>       if [ -n "$1" ]; then
>               curl https://patchwork.freedesktop.org/patch/$1/mbox/ | 
> dim_apply_queued
>       else
>       echo "Give me a patchwork id"
>       fi
> }
>
> v2: Use code-block directive. Get's cleaner and make check happy.

Huh?! For me, this patch leads to:

rst2man --strict --no-raw dim.rst >/dev/null
NotImplementedError: docutils.writers.manpage.Translator visiting unknown node 
type: inline
Exiting due to error.  Use "--traceback" to diagnose.
Please report errors to <[email protected]>.
Include "--traceback" output, Docutils version (0.13.1 [release]),
Python version (2.7.13), your OS type & version, and the
command line used.
Makefile:49: recipe for target 'mancheck' failed
make: *** [mancheck] Error 1

The issue here is that even after the "sphinxification" we need to have
the man pages render using rst2man to have them work properly via 'dim
help'. The ".. code-block::" directive is, AFAICT, a sphinxism.

> Cc: Jani Nikula <[email protected]>
> Cc: Daniel Vetter <[email protected]>
> Signed-off-by: Rodrigo Vivi <[email protected]>
> ---
>  dim.rst | 18 ++++++++++++++++++
>  1 file changed, 18 insertions(+)
>
> diff --git a/dim.rst b/dim.rst
> index 802c776e03f9..d8c9d728dc7b 100644
> --- a/dim.rst
> +++ b/dim.rst
> @@ -441,6 +441,24 @@ usage
>  Short form usage help listing all subcommands. Run by default or if an 
> unknown
>  subcommand was passed on the cmdline.
>  
> +ALIASES
> +=======
> +
> +Extending **dim** functionalities
> +---------------------------------
> +
> +It is possible to create your own dim helper and aliases by adding them to 
> \$HOME/.dimrc
> +
> +.. code-block:: bash
> +
> +             dim_my_fancy_list_aliases()
> +             {
> +                     echo "Hello world!":
> +                     dim_list_aliases:

Why the colons at the end?

BR,
Jani.

> +             }
> +
> +             dim_alias_list_aliases=my-fancy-list-aliases
> +
>  ENVIRONMENT
>  ===========

-- 
Jani Nikula, Intel Open Source Technology Center
_______________________________________________
Intel-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to