Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

"Set file modification date/time of output files to" does not work on Win11 #1661

Closed
5 tasks done
moggaz opened this issue Jul 20, 2023 · 15 comments
Closed
5 tasks done

Comments

@moggaz
Copy link

moggaz commented Jul 20, 2023

I have a lot of issues to go through, so in order to make it easier for me to help you, I ask that you please try these things first

Operating System

Windows 11

Steps to reproduce

Take two or more files, try to merge.

I always use version 3.37.0 because it was the last version where you could still quickly merge files.
Every version after that has to do 6.000.000 steps to merge.
But now that I'm using Windows 11, I noticed that it uses the current time and date when merging even though I set something different. Unfortunately, this does not work even with the current version 3.54.0.

Unfortunately, a quick (2 click, I now need to hit a tiny little symbol....) merge is then also no longer possible :(

Expected behavior

DateTime is now

Actual behavior

DateTime must be datetime of files

Share log

No response

@mifi
Copy link
Owner

mifi commented Jul 21, 2023

Maybe a hotkey would make it easier to merge?

Are you saying you want output file to have modified date set to current clock time?

@moggaz
Copy link
Author

moggaz commented Jul 21, 2023

When I set Windows11 (I think it is default) that when I merge two files it should take the date of the files and not the current date, but it takes the current date. Tested under Windows 10 in a Vm, it works. In Windows 11 it does not.
Hotkey I can try, I did not know yet, thanks.

So the goal is: If I set "take the date of the file" that it then also takes the date of the file and not the current date.

@mifi
Copy link
Owner

mifi commented Jul 24, 2023

Do you see any error message like Failed to set output file modified time pop up in the developer tools log? Unfortunately I don't have windows 11 avail right now so it's hard to look into this issue, but I'm planning on making a github actions windows vm for testing

@moggaz
Copy link
Author

moggaz commented Jul 27, 2023

I do not understand all this but I attached a picture. the source file is from 14.07, today I merge them and although I did not set it in the options, today's date is taken and not that of the source files.

merge

@moggaz
Copy link
Author

moggaz commented Aug 5, 2023

hey, were you able to readjust and fix the error?

@mifi
Copy link
Owner

mifi commented Aug 6, 2023

no, i haven't been able to reproduce the problem, and it works on other platforms

@moggaz
Copy link
Author

moggaz commented Aug 6, 2023

will you fix the error promptly ? otherwise I have to look for a better program that goes with the times (windows 11)...
because at the moment I would destroy all my files

@mifi
Copy link
Owner

mifi commented Aug 6, 2023

it's not so trivial for me to setup a windows 11 vm under apple mac silicon m2, but once i do i will look into it. there's no other way for me to fix this issue without the help of others

@moggaz
Copy link
Author

moggaz commented Aug 6, 2023

i don't get any error message but would like to help because i can't continue until then :(
is it possible that ffmpeg is faulty?

so what can i do?
or is there someone here who knows about this and has windows 11?

@mifi
Copy link
Owner

mifi commented Aug 6, 2023

it's probably not related to ffmpeg.
This is the code that does the job of transferring timestamps:

await fsExtra.utimes(outPath, (atime.getTime() / 1000) + offset, (mtime.getTime() / 1000) + offset);

not sure why it doesn't work for you on win11. I've never heard this issue reported by anyone else.

@mifi
Copy link
Owner

mifi commented Aug 6, 2023

if you know how to run node.js code, you could try to run this code on a file and see if it works.

@mifi
Copy link
Owner

mifi commented Aug 6, 2023

or provide log output from the developer console in losslesscut (more info: https://github.com/mifi/lossless-cut/issues/new?assignees=&labels=&projects=&template=file-issue.yml)

@moggaz
Copy link
Author

moggaz commented Aug 6, 2023

it doesn't seem to work like that, but I've only known node for 2 minutes.
Can you tell me what must be in the js file when I want to read the file 1.mp4?

grafik

@moggaz
Copy link
Author

moggaz commented Aug 16, 2023

anyone here who can help the programmer with this problem ?

@mifi
Copy link
Owner

mifi commented Aug 17, 2023

Ok I've finally managed to install Windows 11 developer preview in a VM on my macbook m2, and when cutting I see that "Modified date" does indeed get correctly set when exporting/merging, see screenshot:

Screenshot 2023-08-17 at 20 31 04

looking at your screenshot however, I can see "Erstelldatum". I believe in english it translates to "created date", so you are not talking about modified date, but "created date". I believe that's not possible to set "created date" of a file in windows, so I'm closing this.

related: nodejs/node#2600

@mifi mifi closed this as not planned Won't fix, can't repro, duplicate, stale Aug 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants