search

toleda / audio_RealtekALC

macOS Realtek ALC onboard audio with Chameleon/Clover
202 stars 75 forks source link

Zipping Scripts? #4

Closed Noctem closed 9 years ago

Noctem commented 9 years ago

Is there a reason that you always zip all of the scripts? It kind of defeats the purpose of a version control system for the commit diffs to just show the removal and addition of zip archives. It would be nice to be able to just track individual changes from the commits without having to compare extracted versions myself locally, or auditing them from scratch all over again.

toleda commented 9 years ago

Objective is the simplest user interaction. The zip file provides a one click download and a double click on the downloaded file to run. If you have a better way, please advise.

drewhammond commented 9 years ago

@toleda Have you thought about using releases? This allows you to tag your repo at various commits and will automatically create .zip and .tar.gz archives for your entire repo at that commit.

You can even attach a prepackaged binary for every release if you wanted to bundle your scripts in a more "one click" way.

But I agree with @Noctem, uploading only zip files to any VCS completely defeats the purpose and discourages other developers to contribute bugfixes and improvements and I filed an issue for the same problem (toleda/audio_CloverALC#3) on another one of your repos a while back.

This solution could be applied to all of your projects and I for one would love to be able to contribute a few things to improve the quality and reliability of your scripts.

Cheers

Noctem commented 9 years ago

Thanks @drewhammond, I was finally coming around to suggest the exact same thing but you got it covered.

toleda commented 9 years ago

audio_realtekALC-100.sh released