Print

Please login or register.
1 Hour 1 Day 1 Week 1 Month Forever
Login with username, password and session length

[Robert Joe]

JRiver Linux - I am using the exact options for the "handheld folder" as in JRiver Windows 19. That is "add cover art to tags" and "copy Folder.jpg"

The Windows produced file when imported into iTunes show Cover Art. But not with Linux.

Strange?
I thought it would be the exact same code base?

Perhaps a bug with case sensitivity folder.jpg vs Folder.jpg? I have not tried renaming it to lowercase, got to leave for work now...

[Robert Joe]

I did a quick test by copying a couple of FLAC files to a temp directory. I forgot to rename Folder.jgp to all lowercase for the test. But the conversion worked with the Cover Art visible in iTunes. The only difference is this temp directory is writable whereas my Music one is read-only. I will test again tonight to see if that is why.

[bob]

I did a quick test by copying a couple of FLAC files to a temp directory. I forgot to rename Folder.jpg to all lowercase for the test. But the conversion worked with the Cover Art visible in iTunes. The only difference is this temp directory is writable whereas my Music one is read-only. I will test again tonight to see if that is why.

It does look to be case sensitive.
We are looking for all lowercase including the extension.
Folder.jpg and folder.jpg and FOLDER.JPG and Folder.JPG and FOLDer.jPg etc etc
are all unique files on nearly all linux filesystems.
I suppose it might be possible to look for the more common variations.

[bob]

The thinking around here is that we should always use the capitalized form going forward.
I.E Folder.jpg Cover.jpg Front.jpg.

Comments?

[Robert Joe]

Mine is Folder.jpg (Only uppercase F) and it works so don't break that one
I think you should just compare case insensitive using strcasecmp().

I did some testing and finally after two hours, here is the bug believe it or not.

    Folder.jpg must have WRITE access e.g. chmod 644 Folder.jpg

If Folder.jpg is read-only, it copies the Folder.jpg over to the destination fine with WRITE access on the copy but the .m4a file imported into iTunes will not display Album Art.

This is reproducible - chmod 444 Folder.jpg then sync/convert breaks it; chmod 644 Folder.jpg then sync/convert it works.

I suspect you are opening Folder.jpg with O_RDWR even though you do not need it. Make it O_RDONLY flag.

[bob]

Mine is Folder.jpg (Only uppercase F) and it works so don't break that one
I think you should just compare case insensitive using strcasecmp().

I did some testing and finally after two hours, here is the bug believe it or not.

    Folder.jpg must have WRITE access e.g. chmod 644 Folder.jpg

If Folder.jpg is read-only, it copies the Folder.jpg over to the destination fine with WRITE access on the copy but the .m4a file imported into iTunes will not display Album Art.

This is reproducible - chmod 444 Folder.jpg then sync/convert breaks it; chmod 644 Folder.jpg then sync/convert it works.

I suspect you are opening Folder.jpg with O_RDWR even though you do not need it. Make it O_RDONLY flag.

Thanks, will check the access mode.
There isn't a filename case independent file open mode on linux (that I'm aware of anyway) so if you really wanted to work like windows/mac you'd have to try opening all combinations of case for every letter in the filename and still you'd end up with the possibility of 2 files coexisting in the same folder, ie Folder.jpg and folder.jpg.