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

Updated make links #2575

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

evan-wallin
Copy link

The link to love-0.10.1 was dead so I updated it to a 0.10.2 link that works. The tmx2lua OSX and Linux code wasn't working in .tar format so I changed it to downloading the .zip and opening from there. It runs perfectly when I run it on my mac or through my emulator, but if there's any issue let me know and I'll work it out.

Copy link
Contributor

@thisIsTheFoxe thisIsTheFoxe left a comment

Choose a reason for hiding this comment

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

Hi, thank you for contributing!

Makefile lgtm, can't test the .ps1, tho...
Also, if you want, you might consider adding your name to the contributors file.

$(wget) https://github.com/hawkthorne/tmx2lua/archive/master.zip
unzip master.zip
rm -rf master.zip
make -C tmx2lua-master
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
make -C tmx2lua-master
make -C tmx2lua-master install
make -C tmx2lua-master

When remaking the file, make sure that go-tmx/tmx is installed.

@niamu
Copy link
Member

niamu commented Dec 16, 2020

This is really great. I just have a couple notes and then I think we can merge this...

I would prefer if we use the GitHub releases links for LÖVE instead of using archive.org

Additionally, we should change the .travis.yml file on line 13 to install 0.10.2 explicitly:
- sudo apt-get install -y love=0.10.2ppa1

Lastly, please see #2555 for how to update the conf file that states which version of LÖVE the game was built for and make that also match 0.10.2

@evan-wallin
Copy link
Author

I think that should take care of it! If there's anything else just lmk

@niamu
Copy link
Member

niamu commented Dec 16, 2020

One last thing I just realized is the need to update line 57 of the Makefile as well to match the version pinning we did in .travis.yml so that we are also fetching the right version of LÖVE for Linux builds.

@niamu
Copy link
Member

niamu commented Dec 17, 2020

My fault for not checking this before I suggested this, but it looks like the PPA doesn't support that older version of LÖVE anymore. Instead we'll need to download and install these packages for Linux builds:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants