BIND 10 #1041: BIND10 build fails if a directory in the path has spaces

BIND 10 Development do-not-reply at isc.org
Fri Aug 26 10:55:48 UTC 2011


#1041: BIND10 build fails if a directory in the path has spaces
-------------------------------------+-------------------------------------
                   Reporter:  cas    |                 Owner:  jelte
                       Type:         |                Status:  reviewing
  defect                             |             Milestone:
                   Priority:  major  |  Sprint-20110830
                  Component:         |            Resolution:
  configuration                      |             Sensitive:  0
                   Keywords:         |           Sub-Project:  Core
            Defect Severity:         |  Estimated Difficulty:  3.0
  Medium                             |           Total Hours:  0
Feature Depending on Ticket:         |
        Add Hours to Ticket:  0      |
                  Internal?:  0      |
-------------------------------------+-------------------------------------
Changes (by vorner):

 * owner:  vorner => jelte


Comment:

 Hello

 I tested it and it works. However, I'd like to point out two things:
  * The test is needlessly complicated. Actually, there's no need to use
 `test` at all, this will work as well:
 {{{
         if echo "$PWD" | grep -q \  ; then
                 ...
         fi
 }}}
    Because -q makes grep silent, only sets the return value if it found
 anything or not.
  * I'd have a proposal for more generic test. The problem isn't only
 spaces, it's any kind of strange characters that make sh act differently.
 So why not test if the path acts differently than it should?
 {{{
         if test "$PWD" != $PWD ; then
                 ...
         fi
 }}}
    It has the problem it also prints a syntax error, though.

 What do you think about it?

 Thank you

-- 
Ticket URL: <http://bind10.isc.org/ticket/1041#comment:13>
BIND 10 Development <http://bind10.isc.org>
BIND 10 Development


More information about the bind10-tickets mailing list