-
Notifications
You must be signed in to change notification settings - Fork 14
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
Build #39
Comments
Travis CIPatchhttps://github.com/john-peterson/dolphin-emu/compare/build4 ProblemA commit should be build tested because
There's no easily accessible build test because Travis CI compared to
pushing to Google Code origin
SolutionAdding a Travis CI script
PlatformInformationThe output from
No multi-platform buildThe build is on either Linux or OS X rather than both The reason to build in Linux is
The reason to build in OS X is
OS XOS X runs less efficiently (compared to Linux) in a virtual machine regarding file access because
debugging because
LinuxSimultaneous make jobsThe make command is
Build timeIRCThe setting
isn't used because
DiscussionBilliard
Sonicadvance1
The travis ci script is described in #39 (comment) Why is
Why don't you want to describe the reason that You should describe the reason because
[03:55] @Sonicadvance1 I'm an unreasonable person I request that you describe the reason for removing delroth
The reply is in the topic "Problem" and "Solution"
The reply is in the topic "Platform"
Approval was
It's not clear if the reply agree with the arguments for the patch in the topics "Problem" and "Solution" because it doesn't reply directly to them
The reply is in the topic GitHub benefit
There's no benefit in fetching from GitHub compared to Google code because
No it only builds in Linux
No, however it can be changed to run any bash command
Even when considering
Complement is a better description than replacement because
If the reason for a change to buildbot is logical it should be implemented
I don't agree that http://buildbot.dolphin-emu.org should be disabled because
Matt_P
There exist arguments n the topic "Problem" and "Solution" that a Travis CI script has value despite the presence of What's your argument that .travis.yml shouldn't be in the repo? [03:15] @Matt_P That is an invalid argument, first provide a reason it should exist to begin with. The reason is in the topic "Problem" and "Solution" in this post
There's a reply to this in the topic "Problem" after the text
[04:03] @Matt_P include it int he test branch [04:04] @Matt_P We currnetly have a build platform. [04:04] @Matt_P If you don't want to use it, then don't. But don't add your random files to the repo.
no you haven't described the cost of having it in the repo that makes the net utility negative describe the cost of having JasperThe arguments from Jasper are paraphrased because the author
RachelB[03:57] @RachelB JPeterson: i think in the time we've spent discussing this, you could have just added it to your clone on github and left it out of the main repo It's described in the topic "Problem" and the "Solution" that it's inefficient
|
OS XLibraryThe pre-build .travis.yml command is
because cmake isn't in the environment These packages are excluded because
is already in the environment
aren't found by cmake
the repo versions are used instead Reference
Package search
|
LinuxReferenceLibraryThe install command is
Package search
Discussion
It runs |
Cygwin
|
|
No description provided.
The text was updated successfully, but these errors were encountered: