Unable to run locally

Bug #1545765 reported by Zisu Andrei
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mail
Fix Released
High
Danielle Foré

Bug Description

Hey guys,

So I've downloaded the Mail app code in an attempt to work on it and I ran into the first problem: the configure script is missing.

I see revision 1950 removed that file, invoking a "legacy build system". Which one is the new one? INSTALL file needs to be updated to reflect this.

I used the Yorba configure file, which succeeded in building the application, but I presume this caused the next problem, when I run the geary binary, I get this:

matzipan@matzipan-thinkpad:~/Workspace/pantheon-mail$ ./geary
** (geary:9158): ERROR **: geary-controller.vala:582: Unable to parse app_menu.interface: Failed to open file '/home/matzipan/Workspace/pantheon-mail/ui/app_menu.interface': No such file or directory
Trace/breakpoint trap (core dumped)

So I guess the "new" build process handles ui files packaging?

Tags: build

Related branches

Revision history for this message
Danielle Foré (danrabbit) wrote :

I've updated the INSTALL file. The build system used here is CMake.

There is a bug where for some reason running locally thinks that ui files should be in a folder that doesn't exist. As a workaround you can link the "data" folder to "ui"

summary: - Build and run problems
+ Unable to run locally
Changed in pantheon-mail:
status: New → Confirmed
importance: Undecided → High
milestone: none → loki-beta1
Changed in pantheon-mail:
status: Confirmed → In Progress
assignee: nobody → Daniel Fore (danrabbit)
Revision history for this message
Zisu Andrei (matzipan) wrote : Re: [Bug 1545765] Re: Build and run problems

Well, the code first looks for the ui folder in /usr/share. If it doesn't
find it there, it looks in the current working directory. I guess we could
use the build process to copy the ui folder in the build folder. Or better
yet, change the code to look for data/ui, although that might require
backporting?

On Tue, 16 Feb 2016, 18:55 Daniel Fore <email address hidden> wrote:

> I've updated the INSTALL file. The build system used here is CMake.
>
> There is a bug where for some reason running locally thinks that ui
> files should be in a folder that doesn't exist. As a workaround you can
> link the "data" folder to "ui"
>
> ** Summary changed:
>
> - Build and run problems
> + Unable to run locally
>
> ** Changed in: pantheon-mail
> Status: New => Confirmed
>
> ** Changed in: pantheon-mail
> Importance: Undecided => High
>
> ** Changed in: pantheon-mail
> Milestone: None => loki-beta1
>
> --
> You received this bug notification because you are subscribed to the bug
> report.
> https://bugs.launchpad.net/bugs/1545765
>
> Title:
> Unable to run locally
>
> Status in Mail:
> Confirmed
>
> Bug description:
> Hey guys,
>
> So I've downloaded the Mail app code in an attempt to work on it and I
> ran into the first problem: the configure script is missing.
>
> I see revision 1950 removed that file, invoking a "legacy build
> system". Which one is the new one? INSTALL file needs to be updated to
> reflect this.
>
> I used the Yorba configure file, which succeeded in building the
> application, but I presume this caused the next problem, when I run
> the geary binary, I get this:
>
> matzipan@matzipan-thinkpad:~/Workspace/pantheon-mail$ ./geary
> ** (geary:9158): ERROR **: geary-controller.vala:582: Unable to parse
> app_menu.interface: Failed to open file
> '/home/matzipan/Workspace/pantheon-mail/ui/app_menu.interface': No such
> file or directory
> Trace/breakpoint trap (core dumped)
>
> So I guess the "new" build process handles ui files packaging?
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/pantheon-mail/+bug/1545765/+subscriptions
>

Revision history for this message
Danielle Foré (danrabbit) wrote :

Hey Zisu, it's been addressed in the linked branch

Revision history for this message
Zisu Andrei (matzipan) wrote : Re: [Bug 1545765] Re: Unable to run locally

Ah, cool.

On Tue, 16 Feb 2016, 21:01 Daniel Fore <email address hidden> wrote:

> Hey Zisu, it's been addressed in the linked branch
>
> --
> You received this bug notification because you are subscribed to the bug
> report.
> https://bugs.launchpad.net/bugs/1545765
>
> Title:
> Unable to run locally
>
> Status in Mail:
> In Progress
>
> Bug description:
> Hey guys,
>
> So I've downloaded the Mail app code in an attempt to work on it and I
> ran into the first problem: the configure script is missing.
>
> I see revision 1950 removed that file, invoking a "legacy build
> system". Which one is the new one? INSTALL file needs to be updated to
> reflect this.
>
> I used the Yorba configure file, which succeeded in building the
> application, but I presume this caused the next problem, when I run
> the geary binary, I get this:
>
> matzipan@matzipan-thinkpad:~/Workspace/pantheon-mail$ ./geary
> ** (geary:9158): ERROR **: geary-controller.vala:582: Unable to parse
> app_menu.interface: Failed to open file
> '/home/matzipan/Workspace/pantheon-mail/ui/app_menu.interface': No such
> file or directory
> Trace/breakpoint trap (core dumped)
>
> So I guess the "new" build process handles ui files packaging?
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/pantheon-mail/+bug/1545765/+subscriptions
>

Changed in pantheon-mail:
status: In Progress → Fix Committed
milestone: loki-beta1 → loki-alpha1
Cody Garver (codygarver)
Changed in pantheon-mail:
status: Fix Committed → Fix Released
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.