[Oorexx-devel] FILE.testGroup ( filesystem case sensitiveness )

2019-02-25 Thread Enrico Sorichetti via Oorexx-devel 

The fragment of code in

ooRexx/base/class/File.testGroup

  subdir = .File~new("Y/Z", f)

  if .File~separator == "\" then  -- verify the separator translation
  self~assertEquals(f~absolutePath||"\Y\Z", subdir~path)

  if \.File~isCaseSensitive then do
  subdir2 = .File~new("y\z", f)
  self~assertEquals(subdir, subdir2)
  end



Is pretty confusing ,

Anyway it fails when the code to detect the filesystem case sensitivity is 
enabled


[failure] [20190225 21:11:21.756899]
  svn:r11775   Change date: 2019-02-17 20:41:37 +0100
  Test:   TEST_RELATIVE
  Class:  File.testGroup
  File:   .../ooRexx/base/class/File.testGroup
  Line:   185
  Failed: assertEquals
Expected: [[/Users/enrico/ooRexx.svn.testSuite//Y/Z], 
identityHash="-4933862865"]
Actual:   [[/Users/enrico/ooRexx.svn.testSuite//y\z], 
identityHash="-4933872305"]


To enable the check 

The relevant lines in
config.h.in.cmake

Should be changed to
/* Define to 1 if _PC_CASE_SENSITIVE is a valid value */
#cmakedefine HAVE_PC_CASE_SENSITIVE


E



___
Oorexx-devel mailing list
Oorexx-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/oorexx-devel

Re: [Oorexx-devel] Testsuite

2019-02-25 Thread Erich Steinböck 
Hi Enrico, this should be fixed with [r11807].
___
Oorexx-devel mailing list
Oorexx-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/oorexx-devel

Re: [Oorexx-devel] Testsuite

2019-02-25 Thread Erich Steinböck 
Hi Enrico,
this should be fixed with revision [r11807].
___
Oorexx-devel mailing list
Oorexx-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/oorexx-devel

Re: [Oorexx-devel] Testsuite

2019-02-25 Thread Chip Davis 

Agreed.

But could you please write the updated code to still allow for an 
"OS/2" possibility, if only for historical (and sentimental) reasons?


-Chip-

On 2/25/2019 6:24 AM, Enrico Sorichetti via Oorexx-devel wrote:


The  code to detect the system is pretty outdated



    -- test for default command processor
    parse source os .                  -- get name of operating system
    os1=os~left(1)~translate           -- get first character in uppercase
    if pos(os1, "O W") > 0 then do     -- OS2, Windows ?
       self~assertSame("CMD", addr, "default address environment for" 
pp(os)"="pp(addr) "not" pp("CMD"))

       end
    else do                            -- all Unix platforms should 
default to "sh"
       self~assertSame("sh", addr, "default address environment for" 
pp(os)"="pp(addr) "not" pp("sh"))

       end




It does not detect OPenBSD

E




___
Oorexx-devel mailing list
Oorexx-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/oorexx-devel

[Oorexx-devel] Testsuite

2019-02-25 Thread Enrico Sorichetti via Oorexx-devel 

The  code to detect the system is pretty outdated



   -- test for default command processor
   parse source os .  -- get name of operating system
   os1=os~left(1)~translate   -- get first character in uppercase
   if pos(os1, "O W") > 0 then do -- OS2, Windows ?
  self~assertSame("CMD", addr, "default address environment for" 
pp(os)"="pp(addr) "not" pp("CMD"))
  end
   else do-- all Unix platforms should default to 
"sh"
  self~assertSame("sh", addr, "default address environment for" 
pp(os)"="pp(addr) "not" pp("sh"))
  end




It does not detect OPenBSD

E



___
Oorexx-devel mailing list
Oorexx-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/oorexx-devel