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

[WIP] Debian packaging #491

Closed
wants to merge 3 commits into from
Closed

[WIP] Debian packaging #491

wants to merge 3 commits into from

Conversation

mbr
Copy link

@mbr mbr commented Dec 19, 2014

This pull requests adds debian packaging.

@mbr
Copy link
Author

mbr commented Dec 19, 2014

I should add a few things: First, this is motivated by repeatedly running into people waiting for instructions on how to install Cockatrice on Linux. Handing out lengthy instructions on how to compile from source are not the answer. =)

Second, it would be nice to have nightly builds automatically on launchpad. I've made this happen on launchpad, purely for proof-of-concept: https://launchpad.net/~m.br/+archive/ubuntu/cockatrice

And finally, I have little interest in maintaining this aspect, as creating debian packages is a real pain. It would be great if someone could pick this up - I'll glady give advice on how to get the automatic builds on launchpads set up; it's rather easy as most of the work is in the debian/rules file.


override_dh_auto_configure:
mkdir $(BUILDDIR)
cd $(BUILDDIR) && cmake .. -DCMAKE_BUILD_TYPE=Release -DWITH_QT4=1 -DWITH_CLIENT=1 -DWITH_SERVER=1 -DWITH_ORACLE=1 -DCMAKE_INSTALL_PREFIX:PATH=/usr
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We don't really want to target Qt4 by default. It would be better to make it use qt5.

For the server I think a multi-build makes sense - cockatrice / cockatrice-server. You can have them both built from the same config

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I believe that is possible, but I'll leave fixing that up to whoever takes over. As mentioned, I'm out =).

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Daenyth what is your thought process behind making it multi-build option? On one hand I would agree options are nice, but for most users it just adds confusion and for the ones that take the time to look into the differences there is a high likely hood they are going to configure things how they choose to rather than rely on pre-defined set of config options.

edit:
Plus, your talking about a very minimal amount of resource overhead to compile both the client and server rather than one or the other.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

By multi-build I mean latest ubuntu desktop + latest ubuntu LTS. If it's in any way a burden I'm fine saying we only support latest version period but I don't think it'll actually be that hard.

The split build is also something that we probably don't need to do

@woogerboy21
Copy link
Contributor

Just my $0.02. If the user isn't willing to put the time in to learn there operating system and how to run applications on it maybe they shouldn't be running the application. I completely disagree with the statement "Handing out lengthy instructions on how to compile from source are not the answer. =)"

Plus im not so sure what 5 lines in linux is a lengthy instruction guide on how to compile from source. If Debian takes more than that and your frustrated at using it, maybe you should try a different distro.

@ZeldaZach
Copy link
Member

Are we going to do anything with this?

@Daenyth
Copy link
Member

Daenyth commented Jan 14, 2015

Yes, I just haven't gotten around to updating it. I need to set up some ubuntu desktop VMs to test with

@Daenyth Daenyth added the OS - Linux Issue specific to linux operating systems label Jan 14, 2015
@ZeldaZach
Copy link
Member

I hope something gets done with this. I hate seeing open PRs that never get looked at x.x

@Daenyth
Copy link
Member

Daenyth commented Jan 26, 2015

I'd like to pick it up but I haven't had time to iterate on it. I can direct you to some resources if you want to learn how to do deb packaging

@ZeldaZach ZeldaZach changed the title Debian packaging [WIP] Debian packaging May 19, 2015
@ZeldaZach
Copy link
Member

I don't see something like this happening in the near future and I think we should close this PR for now

@Daenyth
Copy link
Member

Daenyth commented Jul 14, 2015

It's not hurting anything to have it open. It's a reminder for me to get around to doing it correctly...

On 7/13/15, Zach notifications@github.com wrote:

I don't see something like this happening in the near future and I think we
should close this PR for now


Reply to this email directly or view it on GitHub:
#491 (comment)

@tooomm tooomm mentioned this pull request Oct 13, 2015
@tooomm
Copy link
Member

tooomm commented Oct 17, 2015

What should happen to this after we merged #1646 in?

@Daenyth
Copy link
Member

Daenyth commented Oct 17, 2015

It's no longer needed.

@Daenyth Daenyth closed this Oct 17, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
OS - Linux Issue specific to linux operating systems
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants