[ovs-dev] [PATCH] ovsdb-idlc: Write a new-line at the end of "annotate" output.

Ethan Jackson ethan at nicira.com
Sat Jun 8 20:27:07 UTC 2013


Acked-by: Ethan Jackson <ethan at nicira.com>

On Fri, Jun 7, 2013 at 3:05 PM, Ben Pfaff <blp at nicira.com> wrote:
> Some tools do not like text files that lack a trailing new-line.  In
> particular, Debian's dpkg-source utility complains about a missing new-line
> in the file generated by ovsdb-idlc:
>
>     dpkg-source: warning: file
>     openvswitch-1.9.2+git20130605/lib/vswitch-idl.ovsidl has no final
>     newline (either original or modified version)
>
> This commit fixes the problem.
>
> Signed-off-by: Ben Pfaff <blp at nicira.com>
> ---
>  ovsdb/ovsdb-idlc.in |    1 +
>  1 files changed, 1 insertions(+), 0 deletions(-)
>
> diff --git a/ovsdb/ovsdb-idlc.in b/ovsdb/ovsdb-idlc.in
> index 1f21950..6d49dd6 100755
> --- a/ovsdb/ovsdb-idlc.in
> +++ b/ovsdb/ovsdb-idlc.in
> @@ -18,6 +18,7 @@ def annotateSchema(schemaFile, annotationFile):
>      schemaJson = ovs.json.from_file(schemaFile)
>      execfile(annotationFile, globals(), {"s": schemaJson})
>      ovs.json.to_stream(schemaJson, sys.stdout)
> +    sys.stdout.write('\n')
>
>  def constify(cType, const):
>      if (const and cType.endswith('*') and not cType.endswith('**')):
> --
> 1.7.2.5
>
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev



More information about the dev mailing list