root icon indicating copy to clipboard operation
root copied to clipboard

[core] Introduce suffix in `TSystem::TempFileName()` method

Open linev opened this issue 1 year ago • 16 comments

Allows to specify extension of temporary-created file.

Used in webgui to create temporary html files to start web widgets

linev avatar Feb 20 '24 13:02 linev

Starting build on ROOT-performance-centos8-multicore/soversion, ROOT-ubuntu2204/nortcxxmod, ROOT-ubuntu2004/python3, mac12arm/cxx20, windows10/default How to customize builds

phsft-bot avatar Feb 20 '24 13:02 phsft-bot

Build failed on ROOT-ubuntu2204/nortcxxmod. Running on root-ubuntu-2204-1.cern.ch:/home/sftnight/build/workspace/root-pullrequests-build See console output.

Failing tests:

phsft-bot avatar Feb 20 '24 13:02 phsft-bot

Build failed on ROOT-ubuntu2004/python3. Running on root-ubuntu-2004-1.cern.ch:/home/sftnight/build/workspace/root-pullrequests-build See console output.

Failing tests:

phsft-bot avatar Feb 20 '24 15:02 phsft-bot

Build failed on ROOT-performance-centos8-multicore/soversion. Running on olbdw-01.cern.ch:/data/sftnight/workspace/root-pullrequests-build See console output.

Failing tests:

phsft-bot avatar Feb 20 '24 17:02 phsft-bot

Test Results

    11 files      11 suites   2d 5h 44m 33s :stopwatch:  2 564 tests  2 563 :white_check_mark: 0 :zzz: 1 :x: 26 384 runs  26 382 :white_check_mark: 0 :zzz: 2 :x:

For more details on these failures, see this check.

Results for commit e75e3725.

:recycle: This comment has been updated with latest results.

github-actions[bot] avatar Feb 20 '24 18:02 github-actions[bot]

Starting build on ROOT-performance-centos8-multicore/soversion, ROOT-ubuntu2204/nortcxxmod, ROOT-ubuntu2004/python3, mac12arm/cxx20, windows10/default How to customize builds

phsft-bot avatar Feb 21 '24 08:02 phsft-bot

Build failed on ROOT-performance-centos8-multicore/soversion. Running on olbdw-01.cern.ch:/data/sftnight/workspace/root-pullrequests-build See console output.

Failing tests:

phsft-bot avatar Feb 21 '24 09:02 phsft-bot

Build failed on ROOT-ubuntu2004/python3. Running on root-ubuntu-2004-1.cern.ch:/home/sftnight/build/workspace/root-pullrequests-build See console output.

Failing tests:

phsft-bot avatar Feb 21 '24 09:02 phsft-bot

Looks good to me except the question on the Windows implementation of TempFileName.

Probably rename not needed at all. File not yet exists. One can just will create new file with new name.

linev avatar Feb 21 '24 13:02 linev

Starting build on ROOT-performance-centos8-multicore/soversion, ROOT-ubuntu2204/nortcxxmod, ROOT-ubuntu2004/python3, mac12arm/cxx20, windows10/default How to customize builds

phsft-bot avatar Feb 21 '24 13:02 phsft-bot

Starting build on ROOT-performance-centos8-multicore/soversion, ROOT-ubuntu2204/nortcxxmod, ROOT-ubuntu2004/python3, mac12arm/cxx20, windows10/default How to customize builds

phsft-bot avatar Feb 21 '24 14:02 phsft-bot

Build failed on ROOT-ubuntu2204/nortcxxmod. Running on root-ubuntu-2204-1.cern.ch:/home/sftnight/build/workspace/root-pullrequests-build See console output.

Failing tests:

phsft-bot avatar Feb 21 '24 14:02 phsft-bot

Build failed on ROOT-ubuntu2004/python3. Running on root-ubuntu-2004-1.cern.ch:/home/sftnight/build/workspace/root-pullrequests-build See console output.

Failing tests:

phsft-bot avatar Feb 21 '24 15:02 phsft-bot

Build failed on ROOT-performance-centos8-multicore/soversion. Running on olbdw-01.cern.ch:/data/sftnight/workspace/root-pullrequests-build See console output.

Failing tests:

phsft-bot avatar Feb 21 '24 16:02 phsft-bot

@bellenot

Does this branch works at all on Windows? I mean - starting any web widgets like browser or canvas?

linev avatar Feb 23 '24 10:02 linev

Does this branch works at all on Windows? I mean - starting any web widgets like browser or canvas?

I don't know, I didn't try it. I will try and let you know (but it will take some time...)

bellenot avatar Feb 23 '24 10:02 bellenot

@linev it works. it created a temporary html file as C:\Users\bellenot\AppData\Local\Temp\rooA90E.tmp.html

bellenot avatar Feb 23 '24 12:02 bellenot

it works. it created a temporary html file as C:\Users\bellenot\AppData\Local\Temp\rooA90E.tmp.html

Grate. Do we want more changes in Windows code? Can you approve PR?

linev avatar Feb 23 '24 12:02 linev