Page MenuHomeSolus

wire
Closed, ResolvedPublic

Description

Here is a patch for wire an application for wire.com. Its an electron wrapper so please don't beat me for that.
Its just repackaging the deb, because I don't know how it will handle the api keys if build from official tarball.
wire

Related Objects

StatusAssignedTask
ResolvedJoshStrobl
ResolvedJoshStrobl

Event Timeline

taaem created this task.Dec 9 2016, 4:53 PM
taaem updated the task description. (Show Details)
JoshStrobl closed this task as Wontfix.Dec 9 2016, 4:55 PM
JoshStrobl claimed this task.
JoshStrobl added a subscriber: JoshStrobl.

We don't repackage debs for things that are open source. If it requires an API key, then it should wait until we support private key usage.

taaem added a comment.Dec 9 2016, 5:35 PM

Here is an updated patch, its now building from source and it seems the api keys are not a problem.

JoshStrobl triaged this task as Normal priority.Dec 9 2016, 5:40 PM
JoshStrobl moved this task from Backlog to Awaiting Fixes on the Patch Submission board.

Should look into their actual grunt build target and avoid making a deb only to just extract it, even if it means manually installing files. Also should avoid /opt if possible and patching the desktop file.

taaem added a comment.Dec 10 2016, 1:16 PM

Here is an updated version of the build files, it should be much better now.

JoshStrobl reopened this task as Open.Dec 10 2016, 1:59 PM

%installroot% should change to $installdir and your mkdir -p call should be install, but otherwise looks good!

taaem added a comment.Dec 10 2016, 2:26 PM

Soo this time it should be everything fixed.

Ding, we have a winner.

Unknown Object (User) added a subscriber: Unknown Object (User).Feb 7 2017, 1:45 PM

Doesn't launch on my fresh install as of Feb 7th 2017.

Unknown Object (User) reopened this task as Open.Feb 8 2017, 11:51 AM

I found out that the dependencies libxscrnsaver and gconf were missing. Unfortunately I don't know how to update the package for the Software Center so somebody else will have to do that (or explain to me how to do it/link to a good tutorial)...

JoshStrobl closed this task as Resolved.Feb 8 2017, 1:08 PM

@Dominik Please open up a separate task rather than reopening this one.