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

Building SwiftLint CLI as a standalone/portable file ? #731

Closed
freak4pc opened this issue Jul 25, 2016 · 9 comments
Closed

Building SwiftLint CLI as a standalone/portable file ? #731

freak4pc opened this issue Jul 25, 2016 · 9 comments

Comments

@freak4pc
Copy link
Contributor

Hey guys,
I'm trying to build an integration for CodeClimate that uses SwiftLint.

Their developer platform only allows using Docker and Docker can't run Xcode or OSX so I can't build SwiftLint on the machine.

Would it be possible somehow to "pack" SwiftLint CLI in a standalone format that could be part of a DockerImage ?

Thanks :)
Shai.

@freak4pc freak4pc changed the title Building swiftlint CLI as a standalone/portable file ? Building SwiftLint CLI as a standalone/portable file ? Jul 25, 2016
@freak4pc
Copy link
Contributor Author

Just noticed the Makefile has a portable_zip target. Gonna try that first.

doh

@jpsim
Copy link
Collaborator

jpsim commented Jul 25, 2016

As you mentioned, Docker doesn't support OS X guests, so this would require supporting Linux first. I just created an issue for that (#732). I'll close this one since it's quite a ways away.

@jpsim jpsim closed this as completed Jul 25, 2016
@freak4pc
Copy link
Contributor Author

Would the portable_zip approach be helpful or it won't run properly on a non-OS X environment?

@jpsim
Copy link
Collaborator

jpsim commented Jul 25, 2016

SwiftLint relies on SourceKit, which at the moment isn't included in Linux releases of Swift, so that first needs to be done. So just building the source for SwiftLint isn't enough to make it run on Linux.

@freak4pc
Copy link
Contributor Author

Yeah that was my initial instinct but wanted to make sure.
Thanks a lot !

@vovimayhem
Copy link

There's actually an official Swift image Wouldn't that help?

@jpsim
Copy link
Collaborator

jpsim commented Apr 5, 2017

@vovimayhem the official Swift releases for Linux don't include SourceKit, but swiftlang/swift#8189 will address that.

@caiofbpa
Copy link

@freak4pc SwiftLint seems to fully support Linux now, any news on the Codeclimate Plugin/Engine?

@freak4pc
Copy link
Contributor Author

freak4pc commented Dec 17, 2017

@caiofbpa I have actually dropped this since then and just have a Danger rule. Don't think I'll be pursuing this anytime soon :)

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

No branches or pull requests

4 participants