Menu

#745 SysTempFilename consolidation

5.0.0
closed
nobody
None
none
1
2023-01-01
2019-03-03
Erich
No

The new consolidated SysTempFilename() function returns fully qualified paths (relative to the current directory, of no path is specified).

It used to return /tmp/xxxnnnnnnn on Unix and just the specified file (including path, if any ) on Windows.

Any concerns with that change?
If just a filename ist specified, would it be better to return /tmp/xxxnnnnnnnn (or whatever $TMPNAM says) on Unix and %TEMP%\xxxnnnnnnn on Windows?

Discussion

  • Erich

    Erich - 2019-03-03

    Minor fixes committed with revision [r11819].
    The fix also allows up to 9 filler chars.

     

    Related

    Commit: [r11819]

    • Rick McGuire

      Rick McGuire - 2019-03-03

      These sound like good changes. The shorter filler characters seemed a bit arbitrary to me, so no big deal. Returning a fully qualified name sounds like a good idea to me. On the /tmp/%TEMP% issue, I can make arguments either way. I'm thinking it makes more sense to default to the current directory. One can always explicitly specify the directory if that's where you want to put it. Maybe we should consider adding a tempDirectory method to the .File class so there's a platform-indepdent method of retrieving it.

       
  • Erich

    Erich - 2019-10-04
    • status: unread --> accepted
    • Pending work items: none --> doc+test
     
  • Erich

    Erich - 2019-10-04

    SysFileTempName tests and rexxref updates are still to be done

     
  • Per Olov Jonsson

    Please find attached a testgroup for SysTempFileName and a proposed text for rexxref

     
  • Erich

    Erich - 2020-03-31
    • status: accepted --> pending
    • Pending work items: doc+test --> none
     
  • Erich

    Erich - 2020-03-31

    Added tests and updated rexxref with [r12033]

     

    Related

    Commit: [r12033]

  • Erich

    Erich - 2020-04-01

    Fixed a SysTempFilename test with revision [r12038].

     

    Related

    Commit: [r12038]

  • Rony G. Flatscher

    • Status: pending --> closed
     

Anonymous
Anonymous

Add attachments
Cancel





MongoDB Logo MongoDB