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

Build fails using libtar #928

Closed
sambler opened this issue Aug 17, 2020 · 5 comments
Closed

Build fails using libtar #928

sambler opened this issue Aug 17, 2020 · 5 comments
Assignees
Labels

Comments

@sambler
Copy link

sambler commented Aug 17, 2020

Hydrogen version * : 1.0.0
Operating system + version : FreeBSD 12
Audio driver + version : oss


Setting WANT_LIBARCHIVE=off to build using libtar fails.

I am using libtar v1.2.11 and clang v9.0.1

The error is

/tmp/workdir/hydrogen-1.0.0/src/gui/src/SoundLibrary/SoundLibraryExportDialog.cpp:216:5: 
error: use of undeclared identifier 'TmpFileCreated'
        if(TmpFileCreated)
           ^
1 error generated.

I am unable to find a definition or use for TmpFileCreated anywhere other than that one line in SoundLibraryExportDialog.cpp line 216.

It would appear that the definition was removed in db36fb9

@mauser
Copy link
Member

mauser commented Aug 17, 2020

Oops, my fault :) Should be fixed now in the master branch as well as the 1.0.0 branch.

@mauser mauser self-assigned this Aug 17, 2020
@mauser mauser added the bug label Aug 17, 2020
@elpescado
Copy link
Contributor

Maybe we should add this configuration to CI?

@sambler
Copy link
Author

sambler commented Aug 18, 2020

Fix looks good to me.

My question would be, do you really need to support two archive libraries? libarchive is preferred, why not make it the only compression option?

If you keep two options, I would add it to CI.

@elpescado
Copy link
Contributor

libtar removal was discussed before.

@mauser
Copy link
Member

mauser commented Aug 19, 2020

Hi,

about CI: This is the first bug concerning compilation libtar that i'm aware of. In my opinion it would be a little bit too much to build a libtar-build with every commit. And if we would handle libtar this way, there would be also build with ladspa on/off, liblo on/off and so on. Maybe we could introduce a special branch which has a special travis configuration with a lot of those cmake switches turned on/off?

@mauser mauser closed this as completed Sep 4, 2020
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

3 participants