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

Run compiled binary on Lichee #708

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Run compiled binary on Lichee #708

wants to merge 1 commit into from

Conversation

m0veax
Copy link
Contributor

@m0veax m0veax commented Aug 19, 2023

Hey,

I created this make command together with @orangecms . Someone might find it useful to run the compiled sources on the Lichee. You need to have a usable "xfel" binary in your path for that :)

@codecov
Copy link

codecov bot commented Aug 19, 2023

Codecov Report

Merging #708 (0ab2a05) into main (423914c) will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##            main    #708   +/-   ##
=====================================
  Coverage   1.57%   1.57%           
=====================================
  Files         22      22           
  Lines        954     954           
=====================================
  Hits          15      15           
  Misses       939     939           

@@ -12,6 +12,10 @@ XFEL ?= xfel
MEMADDR ?= 0x40000000
RELEASE ?= --release

MODE ?= release
Copy link
Contributor

Choose a reason for hiding this comment

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

this should be in sync with RELEASE (see above)

How do you feel about putting this into xtask instead? Currently it's a mix already, and gotten messy.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks for the feedback. I'll have a look into xtask and will try to implement that next week

Copy link
Member

Choose a reason for hiding this comment

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

xtask at first looks hard, but you will be happy if you go for it.

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.

None yet

3 participants