subversion-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Stefan Hett (JIRA)" <j...@apache.org>
Subject [jira] [Updated] (SVN-4660) svn info --show-item=revision produces trailing whitespace
Date Tue, 08 Nov 2016 20:44:58 GMT

     [ https://issues.apache.org/jira/browse/SVN-4660?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Stefan Hett updated SVN-4660:
-----------------------------
    Description: 
Steps to reproduce:
1. Run svn info --show-item=revision >out.txt

Expected result:
The file out.txt contains just the revision number and a newline.

Actual result:
The file out.txt contains the revision number, followed by some whitespaces and a newline.

The issue was passed on to the dev list here: http://mail-archives.apache.org/mod_mbox/subversion-dev/201610.mbox/browser

Rational for this being a bug/unintended behavior:
1. One (if not the) main use-case for the newly introduced --show-item option is to make it
easier to be used in scripts. For revisions, I used that to generate a revision range as in:
-rXXX:YYY. XXX was retrieved using --show-item=revision. Since that produced unexpected trailing
whitespaces the script command resulted in [...] -rXXX   :YYY [...] which obviously caused
an error message which was hard to debug since I didn't immediately get the idea that the
trailing whitespaces might cause the script error I saw.
2. It's not what the documentation in the release notes state:
https://subversion.apache.org/docs/release-notes/1.9.html#svn-info-item
where it's explicitly shown that the reported revision number does not have any trailing whitespace:
{noformat}
## Display the youngest revision of a repository:
% svn info --show-item=revision https://svn.apache.org/repos/asf/subversion/trunk
1693514

## Find the root directory of a working copy:
% svn info --show-item=wc-root
/home/jrandom/src/svn/trunk
{noformat}

Note: The issue was not present in SVN < 1.9, since the --show-item-option was added in
SVN 1.9.0. Also I only tested it explicitly with SVN 1.9.4 but since the code change was introduced
in 1.9.0 I'm quite certain the behavior exists in SVN 1.9.0-1.9.3 as well.

  was:
Steps to reproduce:
1. Run svn info --show-item=revision >out.txt

Expected result:
The file out.txt contains just the revision number and a newline.

Actual result:
The file out.txt contains the revision number, followed by some whitespaces and a newline.

The issue was passed on to the dev list here: http://mail-archives.apache.org/mod_mbox/subversion-dev/201610.mbox/browser

Rational for this being a bug/unintended behavior:
1. One (if not the) main use-case for the newly introduced --show-item option is to make it
easier to be used in scripts. For revisions, I used that to generate a revision range as in:
-rXXX:YYY. XXX was retrieved using --show-item=revision. Since that produced unexpected trailing
whitespaces the script command resulted in [...] -rXXX   :YYY [...] which obviously caused
an error message which was hard to debug since I didn't immediately get the idea that the
trailing whitespaces might cause the script error I saw.
2. It's not what the documentation in the release notes state:
https://subversion.apache.org/docs/release-notes/1.9.html#svn-info-item
where it's explicitly shown that the reported revision number does not have any trailing whitespace:
{noformat}
## Display the youngest revision of a repository:
% svn info --show-item=revision https://svn.apache.org/repos/asf/subversion/trunk
1693514

## Find the root directory of a working copy:
% svn info --show-item=wc-root
/home/jrandom/src/svn/trunk
{noformat}


> svn info --show-item=revision produces trailing whitespace
> ----------------------------------------------------------
>
>                 Key: SVN-4660
>                 URL: https://issues.apache.org/jira/browse/SVN-4660
>             Project: Subversion
>          Issue Type: Bug
>          Components: cmdline client
>    Affects Versions: 1.9.4
>            Reporter: Stefan Hett
>
> Steps to reproduce:
> 1. Run svn info --show-item=revision >out.txt
> Expected result:
> The file out.txt contains just the revision number and a newline.
> Actual result:
> The file out.txt contains the revision number, followed by some whitespaces and a newline.
> The issue was passed on to the dev list here: http://mail-archives.apache.org/mod_mbox/subversion-dev/201610.mbox/browser
> Rational for this being a bug/unintended behavior:
> 1. One (if not the) main use-case for the newly introduced --show-item option is to make
it easier to be used in scripts. For revisions, I used that to generate a revision range as
in: -rXXX:YYY. XXX was retrieved using --show-item=revision. Since that produced unexpected
trailing whitespaces the script command resulted in [...] -rXXX   :YYY [...] which obviously
caused an error message which was hard to debug since I didn't immediately get the idea that
the trailing whitespaces might cause the script error I saw.
> 2. It's not what the documentation in the release notes state:
> https://subversion.apache.org/docs/release-notes/1.9.html#svn-info-item
> where it's explicitly shown that the reported revision number does not have any trailing
whitespace:
> {noformat}
> ## Display the youngest revision of a repository:
> % svn info --show-item=revision https://svn.apache.org/repos/asf/subversion/trunk
> 1693514
> ## Find the root directory of a working copy:
> % svn info --show-item=wc-root
> /home/jrandom/src/svn/trunk
> {noformat}
> Note: The issue was not present in SVN < 1.9, since the --show-item-option was added
in SVN 1.9.0. Also I only tested it explicitly with SVN 1.9.4 but since the code change was
introduced in 1.9.0 I'm quite certain the behavior exists in SVN 1.9.0-1.9.3 as well.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message