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

Build Instructions #72

Closed
RowanKaag opened this issue Oct 17, 2016 · 10 comments
Closed

Build Instructions #72

RowanKaag opened this issue Oct 17, 2016 · 10 comments

Comments

@RowanKaag
Copy link

Could anyone provide build instructions? I'm rather new to XCode and the version on https://spotify-notifications.citruspi.io is outdated, nor are there release provided on GitHub.

Thank you in advance!

@citruspi
Copy link
Owner

Once you've got Xcode installed, run

git clone --recursive git@github.com:citruspi/Spotify-Notifications.git

to clone the repository and the git submodules.

Open the Spotify-Notifications.xcodeproj file in Xcode, and then in the menu do Product > Archive.

Hit Export on the right and then select Export as a macOS App unless you've got a Developer Account setup.

untitled

I'm currently traveling and the MacBook I have with me doesn't have my Developer Account set up, but I should be able to build a signed copy and update the website later this week once I get back home.

Until then, here's an unsigned copy in case you can't get the Xcode build running -
Spotify Notifications 2016-10-17 12-00-32.zip

@RowanKaag
Copy link
Author

Thank you, building worked out. I kept getting 'Test' which presumably came from MASShortcut. Changed the 'Product > Scheme' to Spotify-Notifications, working as intended :)

@RowanKaag
Copy link
Author

#53 is working perfectly, thank you so much. Will be posting a video later today.

@RowanKaag
Copy link
Author

RowanKaag commented Oct 18, 2016

What's not immediately clear in the video is that my Spotify window is hidden in the background through changing the LSUIElement. By opening it either through Spotify Notifications or Alfred, it brings up the hidden window. Love this new feature in Spotify-Notifications!

https://www.youtube.com/watch?v=0GFlWA_hZa0

@RowanKaag
Copy link
Author

I can imagine others wanting these instructions. Would it be an idea to incorporate these in the readme.md or make a document like building.md and referende it in the readme?

@sebj
Copy link
Collaborator

sebj commented Oct 19, 2016

Depends on your knowledge of building an app from source, to me at least this all seems pretty familiar stuff. We need to provide the latest release on Github as a pre-built download anyway, and that should be fine for most people.

@citruspi
Copy link
Owner

Depends on your knowledge of building an app from source, to me at least this all seems pretty familiar stuff.

Yeah, I'd agree with that. In general I don't think that the build process for the application is esoteric - it's fairly standard for any macOS or iOS application. Clone the repository (and any dependencies) and then create an archive.

If we provided build instructions, they'd probably look like this:

Clone the repository using

git clone --recursive git@github.com:citruspi/Spotify-Notifications.git

and then build a release.

I may look into creating a Makefile which leverages xcodebuild to build the application for you, but that isn't very high on my to do list to be honest.

We need to provide the latest release on Github as a pre-built download anyway,

Yep, agreed. I was originally supposed to fly back home tonight but my flight was delayed and then moved to tomorrow. Assuming I actually make it back home tomorrow, I should hopefully be able to upload a signed release tomorrow night.

@citruspi
Copy link
Owner

citruspi commented Oct 20, 2016

Another item that's not very high on my to do list, but I might try to setup a spare MacBook as an Xcode build server to perform CI and automatically sign and distribute archives once I get some extended free time.

And then ideally, merge to master would automatically result in a new archive being built, signed, and uploaded and the website being re-built.

@RowanKaag
Copy link
Author

Sounds great, thank you!

@citruspi
Copy link
Owner

Forgot to update the issue, but I cut a release last weekend and updated the website with version 0.6.0.

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

3 participants