Skip to content

OXT-792: OCaml platform -> meta-openxt-ocaml-platform#445

Merged
dpsmith merged 1 commit into
OpenXT:masterfrom
dozylynx:dozylynx-OXT-792-rm-ocaml
Mar 1, 2017
Merged

OXT-792: OCaml platform -> meta-openxt-ocaml-platform#445
dpsmith merged 1 commit into
OpenXT:masterfrom
dozylynx:dozylynx-OXT-792-rm-ocaml

Conversation

@dozylynx

Copy link
Copy Markdown
Contributor

This PR will require a corresponding PR for
meta-openxt-ocaml-platform.git and a new layer definition file for
openxt.git. It will affect build sites due to the need to clone the new
OCaml repository into place for future builds.

Signed-off-by: Christopher Clark christopher.clark6@baesystems.com

This PR will require a corresponding PR for
meta-openxt-ocaml-platform.git and a new layer definition file for
openxt.git. It will affect build sites due to the need to clone the new
OCaml repository into place for future builds.

Signed-off-by: Christopher Clark christopher.clark6@baesystems.com
@dozylynx

Copy link
Copy Markdown
Contributor Author

Please could one of the repository maintainers let me know the next steps to proceed with this and the other related initial stratification PRs for Ocaml and Haskell layers -- thanks.

@jean-edouard

Copy link
Copy Markdown
Member

@rneilturner

Copy link
Copy Markdown
Contributor

It's not good for the image of the community to have PR's unmaintained by the originators and sitting in queue for this long. I would suggest the meta-openxt-ocaml PR's be rejected until such time as they can be properly maintained.

@dozylynx

Copy link
Copy Markdown
Contributor Author

These PRs are in my queue to refresh next week.

@dozylynx

Copy link
Copy Markdown
Contributor Author

From inspection, this PR remains clean to merge along with corresponding openxt PR 203: OpenXT/openxt#203.
This PR conflicts with currently-open xenclient-oe PR #554 .

@dpsmith

dpsmith commented Mar 1, 2017

Copy link
Copy Markdown
Member

lgtm

merging soon

@jean-edouard

Copy link
Copy Markdown
Member

Hmm, did you test this?

@dpsmith

dpsmith commented Mar 1, 2017

Copy link
Copy Markdown
Member

Yes, but you are welcome to do your own build to test as well.

@dpsmith dpsmith merged commit 33224ef into OpenXT:master Mar 1, 2017
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