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

Compiling problem #80

Closed
briefe77 opened this issue Dec 23, 2015 · 2 comments
Closed

Compiling problem #80

briefe77 opened this issue Dec 23, 2015 · 2 comments

Comments

@briefe77
Copy link

root@ubuntu:/home/alexei/Go-For-It-master/build# make
[ 7%] Generating src/Utils.c;src/SettingsManager.c;src/TaskStore.c;src/TaskManager.c;src/view/TaskList.c;src/TaskTimer.c;src/view/TimerView.c;src/view/SettingsDialog.c;src/view/AboutDialog.c;src/view/ContributeDialog.c;src/view/MainWindow.c;src/Main.c
/home/alexei/Go-For-It-master/src/SettingsManager.vala:194.9-194.29: error: The name save_to_file' does not exist in the context ofGLib.KeyFile'
key_file.save_to_file (GOFI.Utils.config_file);
^^^^^^^^^^^^^^^^^^^^^
Compilation failed: 1 error(s), 0 warning(s)
make[2]: *** [go-for-it_valac.stamp] Ошибка 1
make[1]: *** [CMakeFiles/go-for-it.dir/all] Ошибка 2
make: *** [all] Ошибка 2

@JMoerman
Copy link
Collaborator

You are probably using an old valac version with a more recent version of GLib.
I advice you to install a newer version of valac and the corresponding vapi files.

If you insist on building Go For It on your current setup:
remove

# checking if we have a recent version of GLib
pkg_check_modules( GLIB2 glib-2.0 )
if (${GLIB2_VERSION} VERSION_GREATER 2.40 )
    set (VALAC_OPTIONS ${VALAC_OPTIONS} -D HAS_GLIB241)
else ()
    message ("-- No recent glib-2.0 found")
endif ()

from CMakeLists.txt

@JMoerman
Copy link
Collaborator

JMoerman commented Dec 7, 2016

Go for it! now uses GLib.FileUtils.set_contents instead of GLib.KeyFile.save_to_file, so this issue won't occur anymore. (GLib.FileUtils.set_contents has been available since GLib 2.8, which was released in 2005. So it should be available everywhere.)

@JMoerman JMoerman closed this as completed Dec 7, 2016
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

2 participants