#814 date, time, size missing from SysFileTree results

Mark Miesfeld
0 up votes | 0 down votes | 0%

SysFileTree is still documented as returning the date, time, filesize, flags, and filepath/name.
This is the way it has always worked. Now, it never returns date, time, or filesize.

ooRexx Version: 4.0.0
Platform: Linux: Debian 5.0.2a (Lenny) on x86 (32-bit)

This is a new machine, which has never before had any Rexx on it.
Just to be sure, I uninstalled and reinstalled it

dpkg -P ooRexx

dpkg -i ooRexx-4.0.0.i586.deb

Still fails to provide dates, times, or filesizes.

Attached, please find a demonstration program and its output.


<< < 1 2 3 (Page 3 of 3)
  • Mark Miesfeld
    Mark Miesfeld

    On Debian, we see that installing a package actually buitl on a Debian system rather installing a package built on Ubuntu or Kunbuntu resoloves this. I've examined the code several times and the code appears correct.

    As to Fedora, we can not, and have never said we would, support packages not built by the development team and downloaded from SourceForge.

    So, the resolution to this problem is to install the correct package.

  • Mark Miesfeld
    Mark Miesfeld

    I'm opening this back up, just to make a few notes.

    With the re-implementation of SysFileTree I just finished this bug goes away. I built a deb package on a Kubuntu system and one on an Ubuntu system.

    When I run the test program using the older SysFileTree() implementation, the bug still shows, the date time info is missing. When I rerun the test program using the new implementation, no bug. Same results with both the Kubuntu and Ubuntu packages.

    I think Rick was correct when he said:

    Mark, I don't have a Debian system to test on, but this is working ok on
    Fedora. I have a suspicion the problem might be a caused by the sprintf()
    function. The sprintf() call on line 1213 uses ldp-Temp as both the target
    buffer for the formatting and as one of the input arguments. I suspect
    that's causing this to fail.

    Anyhow, it is reassuring to be able to produce the bug and then see it fixed in the current code.

    I'll mark this fixed and pending when I commit.

  • Mark Miesfeld
    Mark Miesfeld

    Committed revision 8126 in trunk

    Testing looks good, but will do more testing before committing to the 4.1 fixes branch

<< < 1 2 3 (Page 3 of 3)


Cancel   Add attachments