PICARD-941: Check Windows Media Player not updating metadata in background
zas
clearly we'll not do that
samj1912
yeah we wont
I was just wondering if we should add a wait for save thread pool to be done
there have been issues about the UI freezing while saving even though we have a seperate save thread
I think it is because they aren't completely seperated?
zas
frankly i don't know ;) i don't experience UI freezes on any on my systems
samj1912
maybe the save thread is blocking the main thread or vice versa?
there was some ticket somewhere
zas
very likely, but it is an issue only when save is slow
samj1912
yeah
that is more of a mutagen issue?
zas
imho we need to find a way to simulate a slow filesystem, and see how Picard behave
because it is very likely issues are hidden when using a fast fs
we have many reports about network shares
samj1912
also luks was of the opinion we should remove the monkey patching for insert bytes and delete bytes
SothoTalKer
might try windows? :)
D4RK-PH0ENiX joined the channel
zas
lol.
samj1912
yeah, the network share problems :\
zas
this monkey patching is useless or no ?
luks said it is now; but are there cases it is still useful ?
about PICARD-941, i wonder if we could simply check for file changes just before saving, and warn user (proposing to reload/merge changes if possible) ?
file monitoring using qfilewatch is a bad idea, i read about it, and basically most implementations just suck (but inotify/linux) and it is very likely to not work well on all platforms
too many limitations (number of files) or performance issues (win/mac)
samj1912
hmmm
we can check for date modified?
SothoTalKer
you ciuld check if file modification time has changed.
samj1912
that seems like the easiest way
zas
to start with, yes
samj1912
also zas, we should remove the compatid3 files and the monkey patching imo
SothoTalKer
maybe also check when picard was in background and has become the active window again?
samj1912
I had a discussion with lazka about compatid3 as well when I was having trouble with TSST/TMOO tags
zas
samj1912: i tend to agree, it looks to me it causes more issues than it solves
samj1912
yeah, plus its hard to isolate the issue, whether its with picard or mutagen :P
so should we remove it now or 1.4.1?
or later?
zas
not sure, looks like too important to do it now, but if it causes serious issues, so i would postpone it to 1.4.1
if it prevents possible file corruption we may include it in 1.4.0
alastairp
praisethemoon: are you downloading the VM image, or building it yourself?
if you're just downloading the image, you don't need to do anything
praisethemoon
alastairp, downloaded the VM
but I can't find pg
alastairp
what are you trying to do?
praisethemoon
connect to the Database from Host to Guest
(Host Being Win10)
samj1912
zas, at max we can show an icon that a file has been modified while it is being saved?
zas
the question is what to do if we detect file changes, for sure we need to prevent any write
alastairp
the VM runs using docker inside the virtual machine, so you won't see a process running on the VM guest
praisethemoon
ah .. so how can I connect to it? ^^
alastairp
there was someone trying this on the forum last week
let me see
praisethemoon
alright, thanks ^^
samj1912
an icon to show it is modified?
zas
samj1912: error icon and log ?
samj1912
yeah
zas
in any case we'll skip any save, we can use the same (hacky) stuff we use in case of network error, having an icon saying "problem" and more infos in Infos..
alastairp
praisethemoon: do you know how to forward ports to the host using virtualbox?
zas
keep it simple, we need first to test it actually changes anything for Sophist
so skip save, and log an error
praisethemoon
alastairp, uhm, not really, but I can google how to do it
alastairp
I'm looking at the configuration for the musicbrainz installation, it looks like port 5432 should be available inside the virtual machine
you should be able to forward that port to your host and connect to it
praisethemoon
alright, gonna try it right away
zas
also note that programs may not change the modification date (ie. Picard has such option), so it will not work in all cases
samj1912
but that forcibly preserves the earlier date right?
for general cases it should work?
praisethemoon
alastairp, isn't user name "musicbrainz" and pwd "musicbrainz"?
zas
i hope so
alastairp
praisethemoon: sorry, I don't know
praisethemoon
okay :) I think it worked, just need to find the pwd