Make implicit dependency on libgee explicit

Bug #1353787 reported by Craig Weber
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Synopsis
Fix Committed
Critical
Mark Raymond Jr.

Bug Description

Synopsis depends on libgee, but CMake doesn't catch the missing dependency because it's not in the CMakeLists.txt file, and `valac` ends up throwing the error when it hits `using Gee;`.

Changed in synopsis:
assignee: nobody → Mark Raymond Jr. (markrtoon)
importance: Undecided → Critical
Revision history for this message
Mark Raymond Jr. (markrjr) wrote :

I'm not seeing this issue on my side. I've got a fresh install of elementary.

I ran sudo apt-get build-dep granite-demo.
Then I ran sudo apt-get install libgranite-dev.

Then I was able to build by creating the build folder, running

cmake -DCMAKE_INSTALL_PREFIX=/usr ../

Then simply run make.

Revision history for this message
Cody Garver (codygarver) wrote : Re: [Bug 1353787] Re: Make implicit dependency on libgee explicit

Because Granite depends on gee, it's getting pulled in for free. I think
this bug report is about adding "gee-0.8" to the CMakeLists.txt as a
dependency so that it's more clearly required.

On Thu, Jul 23, 2015 at 12:43 PM, Mark Raymond Jr. <
<email address hidden>> wrote:

> I'm not seeing this issue on my side. I've got a fresh install of
> elementary.
>
> I ran sudo apt-get build-dep granite-demo.
> Then I ran sudo apt-get install libgranite-dev.
>
> Then I was able to build by creating the build folder, running
>
> cmake -DCMAKE_INSTALL_PREFIX=/usr ../
>
> Then simply run make.
>
> --
> You received this bug notification because you are subscribed to
> elementary.
> https://bugs.launchpad.net/bugs/1353787
>
> Title:
> Make implicit dependency on libgee explicit
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/synopsis/+bug/1353787/+subscriptions
>

--
Cody Garver

Revision history for this message
Mark Raymond Jr. (markrjr) wrote :

Ah, excellent, I've tried adding 'libgee' and 'gee' neither worked.

It built successfully, I'll have the fix up soon.

Thanks!

Changed in synopsis:
status: New → Incomplete
status: Incomplete → In Progress
Changed in synopsis:
status: In Progress → Fix Committed
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.