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

Path to my playlist not remembered (Windows) #10679

Closed
mixxxbot opened this issue Aug 23, 2022 · 15 comments
Closed

Path to my playlist not remembered (Windows) #10679

mixxxbot opened this issue Aug 23, 2022 · 15 comments
Labels
Milestone

Comments

@mixxxbot
Copy link
Collaborator

Reported by: myownshadow
Date: 2022-03-10T18:17:30Z
Status: Fix Released
Importance: Low
Launchpad Issue: lp1964508
Attachments: [220326-MIXXX-Sa, 26.03..csv](https://bugs.launchpad.net/bugs/1964508/+attachment/5570302/+files/220326-MIXXX-Sa, 26.03..csv), mixxx.cfg, mixxx.log.2, mixxx.log.1


I create playlists with an excel programm and save it to a special folder on my laptop.
Playlist import works fine.

But at the next time mixxx has forgotten my path.

I use Version 2.3.2. on Windows
I discussed it already in MIXXX-Forum
https://mixxx.discourse.group/t/path-to-my-playlists/24427/9

Playlist path is empty everytime.

[Library]
EditMetadataSelectedClick 2
EnableWaveformCaching 1
EnableWaveformGenerationWithAnalysis 1
Font MS Shell Dlg 2,10,-1,5,50,0,0,0,0,0,Standard
LastImportExportPlaylistDirectory
RescanOnStartup 1
RowHeight 19
SearchDebouncingTimeoutMillis 302
SeratoMetadataExport 0
ShowBansheeLibrary 0
ShowITunesLibrary 0

@mixxxbot mixxxbot added the bug label Aug 23, 2022
@mixxxbot
Copy link
Collaborator Author

Commented by: ronso0
Date: 2022-03-11T20:54:33Z


maybe a regression of #4531 which wasn't tested on Windows IIRC?

@mixxxbot
Copy link
Collaborator Author

Commented by: ronso0
Date: 2022-03-17T14:33:24Z


I'll prepare a debug build to investigate the issue.

In the meantime, we can try to figure out where the code fails to write the path.
What happens if you set a path manually in the config?
Close Mixxx, open mixxx.cfg, replace

LastImportExportPlaylistDirectory

with

LastImportExportPlaylistDirectory C:\valid\path\to\your\playlists

Is that folder opened when you try to import or export a playlist?
Is the config cleared?

@mixxxbot
Copy link
Collaborator Author

Commented by: ronso0
Date: 2022-03-17T14:38:10Z


Btw when importing/exporting sampler banks saving the directory works the same way.
Could you please test if that path is stored?

In mixxx.cfg it's
[Samplers]
last_import_export_directory ...

@mixxxbot
Copy link
Collaborator Author

Commented by: ronso0
Date: 2022-03-17T20:35:22Z


I opened #4699 to debug this.
See the PR description for details.

@Bernhard Dähnrich
If you don't mind creating a Github account please test this on your machine.
See the PR description for details.

@mixxxbot
Copy link
Collaborator Author

Commented by: myownshadow
Date: 2022-03-18T05:52:12Z
Attachments: [220326-MIXXX-Sa, 26.03..csv](https://bugs.launchpad.net/mixxx/+bug/1964508/+attachment/5570302/+files/220326-MIXXX-Sa, 26.03..csv)


Good Morning !

Sorry guys, but there is a big disaster with your new version
mixxx-2.4-alpha-1130-gc8107cb795-win64.

First of all, the path will still not remenbered
Secondly, no files are importet. Thats new.

I give you my import list as attachment to this eMail.
The export function works fine with your last version.

If you like, you can join me with Teamviewer.
So you can see what happens on my computer.

Thanks for helping.

BMD privat Signatur
Mit freundlichen Grüßen

Bleibt schön gesund !!!
________________________________________________
Bernhard Dähnrich
Tel.: +49 (0) 172 9347698
FAX: +49 (0) 3212 - 1290229
e-Mail: <email address hidden> <mailto:<email address hidden>>
Am 17.03.2022 um 21:35 schrieb ronso0:
> I openedhttps://github.com/mixxxdj/mixxx/pull/4699  to debug this.
> See the PR description for details.
>
> @Bernhard Dähnrich
> If you don't mind creating a Github account please test this on your machine.
> See the PR description for details.
> Instructions for testing:https://github.com/mixxxdj/mixxx/wiki/Testing
>

@mixxxbot
Copy link
Collaborator Author

Commented by: myownshadow
Date: 2022-03-18T07:17:25Z
Attachments: mixxx.cfg, mixxx.log.2, mixxx.log.1


Hello,

additionally to my previous eMail, I send you .log and .cfg files.
I hope, you can find the problem.

BMD privat Signatur
Mit freundlichen Grüßen

Bleibt schön gesund !!!
________________________________________________
Bernhard Dähnrich
Tel.: +49 (0) 172 9347698
FAX: +49 (0) 3212 - 1290229
e-Mail: <email address hidden> <mailto:<email address hidden>>
Am 17.03.2022 um 21:35 schrieb ronso0:
> I openedhttps://github.com/mixxxdj/mixxx/pull/4699  to debug this.
> See the PR description for details.
>
> @Bernhard Dähnrich
> If you don't mind creating a Github account please test this on your machine.
> See the PR description for details.
> Instructions for testing:https://github.com/mixxxdj/mixxx/wiki/Testing
>

@mixxxbot
Copy link
Collaborator Author

Commented by: ronso0
Date: 2022-03-21T21:15:31Z


@Bernhard Dähnrich
Could you please answer my question from comment #⁠3?
Did you already have a chance to test the debug build I posted in #⁠4?

@mixxxbot
Copy link
Collaborator Author

Commented by: myownshadow
Date: 2022-03-23T17:40:43Z


Hallo ronso,

I don't know what I shall do more ?

I tested your Version immediatly and wrote my comment in #⁠5 and #⁠6.

Nothing works.
You got my cfg file from your test-Version.
There you can see that the path will not remembered.

Now I went back to your last stable version 2.3.2. because I need to import a playlist every week.

@mixxxbot
Copy link
Collaborator Author

Commented by: ronso0
Date: 2022-03-23T20:33:34Z


Well, you could actually test the PR build I linked :)

You wrote you were using
mixxx-2.4-alpha-1130-gc8107cb795-win64
which corresponds to the current 'main' branch (nighty build), so i didn't bother looking at the logs, whereas my PR is based on 2.3 and the exact version is
2.3.2-10-g9fc5bbbc54 (playlist-path-debug)

@mixxxbot
Copy link
Collaborator Author

Commented by: myownshadow
Date: 2022-03-24T04:44:28Z


I'm sorry, but do not know exactly which file I have to select for downloading ...

@mixxxbot
Copy link
Collaborator Author

Commented by: ronso0
Date: 2022-03-24T12:55:48Z


I posted all infos in comment #⁠4: read the wiki page (backup & download instructions), create an github account, click the download link, download the binary.
https://bugs.launchpad.net/mixxx/+bug/1964508/comments/4

@mixxxbot
Copy link
Collaborator Author

Commented by: ronso0
Date: 2022-03-24T13:08:13Z


Testing:

  • import to playlists:
    unchanged but produces debug output.
  • importing to crates:
    uses another directory separator, saving the path should work

Please test importing to Crate first. The path is saved to the config as [Library] LastImportExportCrateDirectory
If that works, I'll update playlists, as well.
If it doesn't work, please start a new session and try importing to Playlist. It still won't save the path but it will print debug output, so please upload that session's log.
Thanks!

@mixxxbot
Copy link
Collaborator Author

Commented by: myownshadow
Date: 2022-04-04T06:40:41Z


Good Morning,

sorry, but I was not able to do anything in the last time.

You asked about entrys in mixxx.cfg file.
I cannot find the entry "last_import_export_directory"

[Sampler9]
keylock 0
quantize 0

[Sampler]
LoadSamplerBank 0
SaveSamplerBank 0

[Samplers]
show_expanded 0
show_samplers 1

[Shade]
splitSizeLibTree 568,1346


I added the line manually, but without any effect ..

[Sampler]
LoadSamplerBank 0
SaveSamplerBank 0

[Samplers]
last_import_export_directory D:\LineDance\Tänze\Playlists
show_expanded 0
show_samplers 1

[Shade]
splitSizeLibTree 568,1346

BMD privat Signatur
Mit freundlichen Grüßen

Bleibt schön gesund !!!
________________________________________________
Bernhard Dähnrich
Tel.: +49 (0) 172 9347698
FAX: +49 (0) 3212 - 1290229
e-Mail: <email address hidden> <mailto:<email address hidden>>
Am 17.03.2022 um 15:38 schrieb ronso0:
> Btw when importing/exporting sampler banks saving the directory works the same way.
> Could you please test if that path is stored?
>
> In mixxx.cfg it's
> [Samplers]
> last_import_export_directory ...
>

@mixxxbot
Copy link
Collaborator Author

Commented by: ronso0
Date: 2022-04-10T12:49:01Z


Issue confirmed on Win10

I just updated #4699
Confrimed using "/" as directory separator on all OS fixes the issue.

@mixxxbot
Copy link
Collaborator Author

Issue closed with status Fix Released.

@mixxxbot mixxxbot transferred this issue from another repository Aug 24, 2022
@mixxxbot mixxxbot added this to the 2.3.3 milestone Aug 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant