-
Notifications
You must be signed in to change notification settings - Fork 13
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
Please merge my various changes and fixes #4
Open
michaelknigge
wants to merge
52
commits into
sobomax:master
Choose a base branch
from
michaelknigge:master
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…w64-mingw32-windres".
…er - and not the Shell-Skript "do-test.sh". As a result, Travis-CI no longer builds the Windows version of digger. The Windows-Version is built on AppVeyor.
…en mode using F11.
…e on www.digger.org is completely outdated.
…ous releases the Hobbin came first in as a Nobbin and then, after a short time, mutated to a Hobbin. This was no bug - it was implemented this way. But from the users view it looked like a bug (and the 1983 Digger worked as it is implemented now).
…before the right part of the title screen gets blanked. Those effects looked like a gfx bug/glitch so I have removed them. Even the 1983 Digger did not show an effect like this.
…sampling rate and buffer size may sill be set in the DIGGER.INI file. I guess this is useless, too.... but who knows.
… be specified in the DIGGER.INI file. Currently if they are presend, they are considered. But there are also command line options that assign values to this settings. So... who should win? If the DIGGER.INI conatins "SoundOn=false", how should th user turn the sounds on? If the DIGGER.INI conatains "Players=2", how to play in single user mode? So let's ignore those settings. This also makes the command line options "/G" and "/2" work again.
…mes in as a Hobbin again and the "effects" before the right half of the screen gets blanked are removed again.
…er mode. Instead the game toggles between all possible game modes.
…r Linux *AND* Windows on Travis-CI (running Linux) again. On AppVeyor Digger is also built for Windows using Visual Studio. Note that the executable of the utility windres is named differently on Linux and Windows. We use the "Linux-Name" in the Makefile and override the name for Windows in the file appveyor.yml
…elease under the GPL (see http://digger.org/faq.html) in 1998. Since the GPL v3 was released 2007, I guess that GPLv2 is the correct licence.
sobomax
force-pushed
the
master
branch
16 times, most recently
from
December 28, 2022 01:09
6131760
to
5fcdbe4
Compare
sobomax
force-pushed
the
master
branch
13 times, most recently
from
January 4, 2023 22:51
9eaa545
to
728c22a
Compare
Hey @michaelknigge it's just a bit ball of changes to injest. Perhaps splitting up into a smaller bits would make it bit easy to review and merge. Thanks! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.