-
Notifications
You must be signed in to change notification settings - Fork 58
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
Update build guide #134
Update build guide #134
Conversation
xihuan-citrix
commented
Nov 16, 2022
- Update README.md
- Move dependency on the project name when building
bf086fd
to
b2fff77
Compare
b2fff77
to
46a4dcb
Compare
README.md
Outdated
`GO111MODULE` | ||
Set `GO111MODULE` disabled | ||
e.g. | ||
let's say you project directory is /home/xe-guest-utilities-7.30.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
your
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changed
README.md
Outdated
5. Build | ||
Go into the right directory `cd $GOPATH/src/github.com/xenserver/xe-guest-utilities` | ||
now you can `make build` or `make`. | ||
resulting files are in `build/`, same layout as explained below | ||
----------- | ||
Type `make` or `make build` to build the xenstore and xe-daemon. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This line should merge with line 75
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changed
README.md
Outdated
With `GO111MODULE` enabled, go projects are no longer confined to $GOPATH, instead it use go.mod to keep track fo each package and it's version | ||
|
||
e.g. | ||
let's say you project directory is /home/xe-guest-utilities-7.30.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
your
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changed
mk/xe-linux-distribution
Outdated
fi | ||
|
||
eval $(sed -rn \ | ||
's/^Kylin Linux Advanced Server release V([0-9][0-9]).*$/distro=kylin;major=\1;/gp;' \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
V([0-9][0-9])
could be V([0-9]{2})
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's a mistake in the commit, I'v merged all of the commit into one. Thus it will not show this modification anymore. Previous one commit modify this, another commit recover this. I do this modification to test Kylin V10, it should not exist.
build Signed-off-by: Xihuan Yang <xihuan.yang@citrix.com>
46a4dcb
to
d9852c4
Compare