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

Update detectdaterangedialog.cpp #1

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

fl0f
Copy link

@fl0f fl0f commented May 26, 2016

Trying to compile with linux system : build-eddypro_win-Desktop_Qt_5_5_1_GCC_64bit-Debug
An error occured at line 509. I assumed "auto" variable types (defined at lines 503 and 506) were not correct and changed them to "QString"
I did the same for the other cases of function showDateRange lines 528, 531, 553, 556, 578, 581.

I then could make my first runs with EddyPro on linux !!!

Best
Florent

Trying to compile with linux system : build-eddypro_win-Desktop_Qt_5_5_1_GCC_64bit-Debug
An error occured at line 509. I assumed "auto" variable types (defined at lines 503 and 506) were not correct and changed them to "QString"
I did the same for the other cases of function showDateRange lines 528, 531, 553, 556, 578, 581.

I then could make my first runs with EddyPro on linux !!!

Best
Florent
@bddbdd
Copy link

bddbdd commented May 26, 2016

I think the 'auto' forms are for newer C++, specifically letting the compiler determine data type.

What compiler and version are you using?

@fl0f
Copy link
Author

fl0f commented May 26, 2016

Hi,

OK, maybe there is little incompatibility between our versions of Qt
libraries.

I'm using that :

Qt Creator 3.6.0 Basé sur Qt 5.5.1 (GCC 4.9.1 20140922 (Red Hat
4.9.1-10), 64 bit) Built on Dec 15 2015 01:01:12 From revision b52c2f91f5

Florent

Le 26/05/2016 13:44, bddbdd a écrit :

I think the 'auto' forms are for newer C++, specifically letting the
compiler determine data type.

What compiler and version are you using?

— You are receiving this because you authored the thread. Reply to
this email directly or view it on GitHub
#1 (comment)

@emiola
Copy link
Contributor

emiola commented Aug 24, 2016

Hi Florent, I apologize for the really late reply.

I just published a new version with explicit support for Linux. I'm currently able to compile and run on Ubuntu and Fedora/CentOS. I will add in the README more information on how to compile on Linux, but I can also help you directly to figure out details.

Anyway, @bddbdd is right: auto is a c++11 keyword and requires a compiler supporting this standard. I'm sure we can find a solution. Please, provide compiler and version you are currently using. Thanks,

Antonio

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

Successfully merging this pull request may close these issues.

3 participants