-
Notifications
You must be signed in to change notification settings - Fork 162
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
Consider distribution via custom Homebrew Tap #32
Comments
TL;TR of our private discussion of this thing: I'm personally kinda not sure about that, we'll have to be very conservative once we release Good example of such a problem is |
@ming13 you can probably contribute formula into central repo |
@thevery hello there lol 😸, surely we can, but it's more a question of "should we"? I see this very similar to things like Another point is that team members don't have to install anything to make it work once it in the project repo, while with Moreover, once we publish |
I like the idea of having sth like mainframer wrapper which will work analogically to In such aprroach you don't force users to update but still they can do it very easly. |
Yup. |
I was thinking about something else. |
Hm, that's interesting, let us think about it for couple days
…On Fri, Jan 27, 2017, 19:40 Kasper Kondzielski ***@***.***> wrote:
I was thinking about something else.
The mainframerWrapper could delegate all normal invocations to
mainframer.sh, but in addition it could check some
mainframerWrapper.properties file (where there version of mainframer
would be defiend) and update mainframer.sh if needed.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#32 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AA7B3JneXKSk2akefQC-6K2-7TOsTMr-ks5rWh3pgaJpZM4LUCvK>
.
|
@artem-zinnatullin I was thinking about writing Mainframer in Go. It's easy to provide binaries for every system that way. Making backwards-compatible changes are also a lot easier because of all the benefits you get when using almost any programming language. There would also only be a single binary. |
That would require us to make ssh and rsync compatible clients which is
much more complicated than current version of Mainframer. Or you want to
call ssh and rsync from it? If so, I'm not sure benefits worth it %)
Also binary will be heavier than text file in terms of size and would
require to make some kind of wrapper (like gradlew) which basically removes
all the benefits of rewriting Mainframer from bash to other languages since
we'll still need some shell.
…On Sat, Apr 15, 2017, 17:05 Andreas Backx ***@***.***> wrote:
@artem-zinnatullin <https://github.com/artem-zinnatullin> I was thinking
about writing Mainframer in Go. It's easy to provide binaries for every
system that way and making backwards-compatible changes are a lot easier
because of all the benefits you get when using almost any programming
language. There would also only be a single binary.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#32 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AA7B3CCvqyXb4s34mlRmIxIQHcMW6mUEks5rwM6ygaJpZM4LUCvK>
.
|
For a simple port I'd start with simply calling ssh and rsync from it. In the future we could implement these directly using Go libraries and the benefit would then be that no setup is necessary on the client machine and it could be made cross platform. We could have I do think that the size of binary is negligible, it would then be system-wide and not per-project. Could you explain why you would want to create a wrapper for mainframer when using a binary? I don't see a mainframer as an essential part of a project. |
Update: we're considering publishing Mainframer to brew and other repos in some near future since tool is so mature and stable now that it's hard to work on projects without it (it's like git/etc). |
No description provided.
The text was updated successfully, but these errors were encountered: