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

Util.h not found #5

Open
pierrep opened this issue Aug 15, 2013 · 11 comments
Open

Util.h not found #5

pierrep opened this issue Aug 15, 2013 · 11 comments

Comments

@pierrep
Copy link

pierrep commented Aug 15, 2013

I'm compiling for Linux at the moment, and I'm getting this error in LFNoise.cpp due to Util.h not being found:

ifndef _WIN32

include "Util.h"

endif

I just commented it out and that solves the problem. What's the use for this header exactly? Or am I just picking up unfinished code that's not included in the XCode project?

@ndonald2
Copy link
Member

I have a feeling that's a leftover that never got removed. Seems safe to take out.

@morganpackard Any insights?

@morganpackard
Copy link
Member

Yeah, leftover I think. If I recall correctly, LFNoise is not done, and
slipped in by accident. It might be buggy and/or broken.

On Wed, Aug 14, 2013 at 10:12 PM, Nick D. [email protected] wrote:

I have a feeling that's a leftover that never got removed. Seems safe to
take out.

@morganpackard https://github.com/morganpackard Any insights?


Reply to this email directly or view it on GitHubhttps://github.com//issues/5#issuecomment-22681157
.

Morgan Packard
cell: (720) 891-0122
twitter: @morganpackard

@morganpackard
Copy link
Member

We haven't tested much on Linux. Let please let us know how it goes for you.

@pierrep
Copy link
Author

pierrep commented Aug 22, 2013

Yeah sure, I got it to compile but had other issues then dropped what I was doing. I'm happy to send a PR with a Linux version if that's useful.

@morganpackard
Copy link
Member

@pierrep We have a linux branch in the main repo that we've just never merged in. https://github.com/TonicAudio/Tonic/commits/ND-Linux If you see Tonic being useful to you, we'd love a pull request, but it would need to be to the main Tonic repository: https://github.com/TonicAudio/Tonic , not ofxTonic.

Please take a look at our VCS guidelines before you give the pull request.
https://github.com/TonicAudio/Tonic/wiki/Collaboration-and-Version-Control-Guidelines

@pierrep
Copy link
Author

pierrep commented Sep 24, 2013

I've just revisited this, I'm using the ND-Linux branch and it seemed to solve all the previous problems. I've compiled Tonic as a static library as well, but I'm getting a new error in ofxTonicSynth that I didn't seem to get before:

../../../addons/ofxTonic/src/ofxTonicSynth.cpp|31|error: ‘class Tonic::ControlGenerator’ has no member named ‘addValueChangedSubscriber’|

Any idea how to fix this?

@morganpackard
Copy link
Member

Which branch are you working on? I'm guessing what you did was manually
copy over files from the ND-Linux branch. Is that correct? That branch is
old and needs some of the new stuff merged in to it. Would you be up for
making your own linux branch (branched off of ND-Linux), and merging our
development branch into it? If that merge goes smoothly, it should solve
your problems, and then you can submit a pull request when it's working.

Thanks!

On Tue, Sep 24, 2013 at 4:00 PM, Pierre Proske [email protected]:

I've just revisited this, I'm using the ND-Linux branch and it seemed to
solve all the previous problems. I've compiled Tonic as a static library as
well, but I'm getting a new error in ofxTonicSynth that I didn't seem to
get before:

../../../addons/ofxTonic/src/ofxTonicSynth.cpp|31|error: ‘class
Tonic::ControlGenerator’ has no member named ‘addValueChangedSubscriber’|

Any idea how to fix this?


Reply to this email directly or view it on GitHubhttps://github.com//issues/5#issuecomment-25037412
.

Morgan Packard
cell: (720) 891-0122
twitter: @morganpackard

@pierrep
Copy link
Author

pierrep commented Sep 24, 2013

I cloned Tonic and checked out the ND-Linux branch. I was actually going to suggest a merge...not sure if I have time for it right now, I actually want to go and build some sound stuff instead :)

I just commented that event stuff out, everything else seems to work. Omg so far the synths sound AWESOME! This is from someone with lots of experience with analogues and VA's/soft synths.

But yes, I will try and merge with the latest stuff, I don't like the idea of running old code...

@morganpackard
Copy link
Member

Great! Thanks!
We're planning on doing an official release in the next month or so, and
getting that linux stuff in there. But having help from some who actually
USES linux would be very much appreciated.

On Tue, Sep 24, 2013 at 5:26 PM, Pierre Proske [email protected]:

I cloned Tonic and checked out the ND-Linux branch. I was actually going
to suggest a merge...not sure if I have time for it right now, I actually
want to go and build some sound stuff instead :)

I just commented that event stuff out, everything else seems to work. Omg
so far the synths sound AWESOME! This is from someone with lots of
experience with analogues and VA's/soft synths.

But yes, I will try and merge with the latest stuff, I don't like the idea
of running old code...


Reply to this email directly or view it on GitHubhttps://github.com//issues/5#issuecomment-25044003
.

Morgan Packard
cell: (720) 891-0122
twitter: @morganpackard

@pierrep
Copy link
Author

pierrep commented Sep 24, 2013

I hear you. I might just have a bash at it now. I have a gig this Saturday, hoping to sneak some Tonic in there...

@pierrep
Copy link
Author

pierrep commented Sep 24, 2013

By the way merging the Linux branch into development fixed the problems I was having above, just as you said.

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

3 participants