#814 date, time, size missing from SysFileTree results

4.1.3
closed
Mark Miesfeld
none
5
2013-07-09
2009-09-13
D.M.Wooster
No
0 up votes | 0 down votes | 0%
26 comments

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.

Discussion

<< < 1 2 3 (Page 3 of 3)
  • Mark Miesfeld
    Mark Miesfeld
    2012-02-03

    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
    2012-07-30

    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
    2012-07-30

    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)


Anonymous


Cancel   Add attachments