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

Switch to single binary, and some makefile cleanup #3

Merged
merged 2 commits into from
Apr 7, 2020

Conversation

deitch
Copy link
Contributor

@deitch deitch commented Apr 7, 2020

As discussed in #1

This PR does a few things:

  • switches from multiple commands to a single command
  • updates the references in the Makefile and the README to use that single command (I might have missed one or two; please do review)
  • does some makefile cleanup of shell commands and verbosity to make it easier to use
  • changes the output file to dist/bin/eden-<os>-<arch> so we can cross-build with ease
  • adds a symlink in the root to the binary for your os+arch so you can just run ./eden

@sadov
Copy link
Contributor

sadov commented Apr 7, 2020

Thanks @deitch -- we'll look.

Copy link
Collaborator

@giggsoff giggsoff left a comment

Choose a reason for hiding this comment

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

Thanks @deitch -- I left few comments below:

Makefile Outdated Show resolved Hide resolved
cmd/server.go Outdated Show resolved Hide resolved
cmd/certs.go Outdated Show resolved Hide resolved
@deitch
Copy link
Contributor Author

deitch commented Apr 7, 2020

Great. I will hit those right away.

Signed-off-by: Avi Deitcher <avi@deitcher.net>
@deitch
Copy link
Contributor Author

deitch commented Apr 7, 2020

All updated.

@giggsoff
Copy link
Collaborator

giggsoff commented Apr 7, 2020

Looks great, thanks a lot @deitch!

@deitch
Copy link
Contributor Author

deitch commented Apr 7, 2020

We need @rvs to pull it in. I have merge rights on Adam, not lf-edge org in general.

@deitch deitch requested a review from rvs April 7, 2020 17:47
Copy link
Contributor

@rvs rvs left a comment

Choose a reason for hiding this comment

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

LGTM!

@rvs rvs merged commit 4473af4 into lf-edge:master Apr 7, 2020
@rvs
Copy link
Contributor

rvs commented Apr 7, 2020

Hey @deitch you can now merge changes into the repo as well.

I need to setup a proper committer file to add everybody else on the team tho

@deitch deitch deleted the single-command branch April 8, 2020 08:52
@deitch
Copy link
Contributor Author

deitch commented Apr 8, 2020

Thanks @rvs

rvs pushed a commit to rvs/eden that referenced this pull request Apr 30, 2020
mydatascience pushed a commit that referenced this pull request Jan 11, 2021
mydatascience added a commit that referenced this pull request Feb 5, 2021
* # This is a combination of 7 commits.
# This is the 1st commit message:

Added setup action

Signed-off-by: fleandr <svfly@yandex.ru>

# This is the commit message #2:

Fix setup action

Signed-off-by: fleandr <svfly@yandex.ru>

# This is the commit message #3:

Fix setup action

Signed-off-by: fleandr <svfly@yandex.ru>

# This is the commit message #4:

Fix setup action

Signed-off-by: fleandr <svfly@yandex.ru>

# This is the commit message #5:

Fix setup action

Signed-off-by: fleandr <svfly@yandex.ru>

# This is the commit message #6:

Fix setup action

Signed-off-by: fleandr <svfly@yandex.ru>

# This is the commit message #7:

Fix setup action

Signed-off-by: fleandr <svfly@yandex.ru>

* Fix setup action

Signed-off-by: fleandr <svfly@yandex.ru>

* Fix setup action

Signed-off-by: fleandr <svfly@yandex.ru>

* Fix setup action

Signed-off-by: fleandr <svfly@yandex.ru>
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.

4 participants