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

Add github action to build juicefs and libjfs.so #2

Merged
merged 8 commits into from
Aug 17, 2021
Merged

Add github action to build juicefs and libjfs.so #2

merged 8 commits into from
Aug 17, 2021

Conversation

HugoAhoy
Copy link
Contributor

@HugoAhoy HugoAhoy commented Aug 9, 2021

Added github action to build juicefs and libjfs.so.
The juicefs and libjfs.so are stored in cache, which will speed up the ci.
Updated the README. Added how to build juicefs and libjfs.so if users build from source.

Copy link
Contributor Author

@HugoAhoy HugoAhoy left a comment

Choose a reason for hiding this comment

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

fix a typo

README.md Outdated
@@ -32,10 +32,17 @@ Clone the repository:
git clone git@github.com:megvii-research/juicefs-python.git
```

And then install requirements:
And then build `juicefs` ans `libjfs.so`:
Copy link
Contributor Author

Choose a reason for hiding this comment

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

'ans' -> 'and'

@bbtfr
Copy link
Collaborator

bbtfr commented Aug 17, 2021

👍

@bbtfr bbtfr merged commit b3aa154 into megvii-research:main Aug 17, 2021
bbtfr pushed a commit that referenced this pull request Aug 30, 2021
* Add release (#1)

* add release.yaml to help release

* change the action name

* delete trigger branch limit

* add branch limit

* Add release (#2)

* add release.yaml to help release

* change the action name

* delete trigger branch limit

* add branch limit

* add needs

* fix some bugs

* add wheel to requirements-dev.txt

* fix wrong command

* move publish to a single job

* delete wrong command

* change upload mistake

* fix wrong macos platform tag

* juicefs can format and init on Windows

* Wrap os.path.join() with Path().as_posix() to support Windows. jfs.concat() remains problematic on Windows

* format the files
bbtfr pushed a commit that referenced this pull request Aug 30, 2021
* Add release (#1)

* add release.yaml to help release

* change the action name

* delete trigger branch limit

* add branch limit

* Add release (#2)

* add release.yaml to help release

* change the action name

* delete trigger branch limit

* add branch limit

* add needs

* fix some bugs

* add wheel to requirements-dev.txt

* fix wrong command

* move publish to a single job

* delete wrong command

* change upload mistake

* fix wrong macos platform tag

* add win build and test action

* fix syntax error

* enable download artifact on windows

* Add win compatibility (#3)

* juicefs can format and init on Windows

* Wrap os.path.join() with Path().as_posix() to support Windows. jfs.concat() remains problematic on Windows

* format the files

* add win build and test action

* fix syntax error

* enable download artifact on windows

* add windows build and test to on-push.yaml

* change the build win binary if statements

* remove win_build_test.yaml. It's merged into on-push.yaml
bbtfr pushed a commit that referenced this pull request Aug 30, 2021
* Add release (#1)

* add release.yaml to help release

* change the action name

* delete trigger branch limit

* add branch limit

* Add release (#2)

* add release.yaml to help release

* change the action name

* delete trigger branch limit

* add branch limit

* add needs

* fix some bugs

* add wheel to requirements-dev.txt

* fix wrong command

* move publish to a single job

* delete wrong command

* change upload mistake

* fix wrong macos platform tag

* Add win compatibility (#3)

* juicefs can format and init on Windows

* Wrap os.path.join() with Path().as_posix() to support Windows. jfs.concat() remains problematic on Windows

* format the files

* Win build (#4)

* add win build and test action

* fix syntax error

* enable download artifact on windows

* add win build and test action

* fix syntax error

* enable download artifact on windows

* add windows build and test to on-push.yaml

* change the build win binary if statements

* remove win_build_test.yaml. It's merged into on-push.yaml

* add windows build and publish to release.yaml

* update requirements-dev.txt to support python3.5

* fix requirements-dev.txt syntax error

* delete the f-string because python3.5 doesn't support it

* only test python3.5 in test

* make variable annotation compatible with py35

* make variable annotation compatible with py35

* make variable annotation compatible with py35

* format the file

* add py36, py37, py38, py39 to the ci build and test
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

2 participants