Skip to content

Remove solaris build tag and contrib/mkimage/solaris#35373

Merged
AkihiroSuda merged 1 commit intomoby:masterfrom
yongtang:11012017-solaris
Nov 2, 2017
Merged

Remove solaris build tag and contrib/mkimage/solaris#35373
AkihiroSuda merged 1 commit intomoby:masterfrom
yongtang:11012017-solaris

Conversation

@yongtang
Copy link
Member

@yongtang yongtang commented Nov 1, 2017

As Solaris is not really supported any more. I am proposing to remove the build tags and other reference to Solaris from the code base.

This fix cleaned all traces of solaris in build tags and files.

Signed-off-by: Yong Tang [email protected]

Copy link
Member

@tianon tianon left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

Choose a reason for hiding this comment

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

this file should probably be renamed to _linux.go then the build-tag can be removed

Copy link
Member Author

Choose a reason for hiding this comment

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

Thanks that works! Updated.

Copy link
Member

Choose a reason for hiding this comment

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

same here

Copy link
Member Author

Choose a reason for hiding this comment

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

Done.

Copy link
Member

Choose a reason for hiding this comment

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

same here

Copy link
Member Author

Choose a reason for hiding this comment

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

Done.

Copy link
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

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

LGTM (if green 😄)

@AkihiroSuda AkihiroSuda merged commit 76531cc into moby:master Nov 2, 2017
@yongtang yongtang deleted the 11012017-solaris branch November 3, 2017 19:08
kolyshkin pushed a commit that referenced this pull request Sep 2, 2019
Support for GOOS=solaris was removed in PR #35373. Remove two leftover
*_solaris.go files missed in this PR.

Signed-off-by: Tobias Klauser <[email protected]>
burnMyDread pushed a commit to burnMyDread/moby that referenced this pull request Oct 21, 2019
Support for GOOS=solaris was removed in PR moby#35373. Remove two leftover
*_solaris.go files missed in this PR.

Signed-off-by: Tobias Klauser <[email protected]>
Signed-off-by: zach <[email protected]>
cipherboy added a commit to cipherboy/openbao that referenced this pull request Nov 14, 2024
Solaris has no support in upstream Moby any more; while we currently
only test with Docker, we may wish to support containerized plugins
running on remote hosts in the future and would be unable to do so from
Solaris.

Additionally, this release target sees few downloads and thus isn't
worth maintainer time to support.

See also: moby/moby#35373
See also: https://tooomm.github.io/github-release-stats/?username=openbao&repository=openbao

Signed-off-by: Alexander Scheel <[email protected]>
cipherboy added a commit to cipherboy/openbao that referenced this pull request Nov 14, 2024
Solaris has no support in upstream Moby any more; while we currently
only test with Docker, we may wish to support containerized plugins
running on remote hosts in the future and would be unable to do so from
Solaris.

Additionally, this release target sees few downloads and thus isn't
worth maintainer time to support.

See also: moby/moby#35373
See also: https://tooomm.github.io/github-release-stats/?username=openbao&repository=openbao

Signed-off-by: Alexander Scheel <[email protected]>
cipherboy added a commit to openbao/openbao that referenced this pull request Nov 14, 2024
* Fully upgrade Docker dependency

This follows churn caused by Docker removing type aliases in use by
others.

See also: moby/moby#47148

Signed-off-by: Alexander Scheel <[email protected]>

* Remove support for Solaris builds

Solaris has no support in upstream Moby any more; while we currently
only test with Docker, we may wish to support containerized plugins
running on remote hosts in the future and would be unable to do so from
Solaris.

Additionally, this release target sees few downloads and thus isn't
worth maintainer time to support.

See also: moby/moby#35373
See also: https://tooomm.github.io/github-release-stats/?username=openbao&repository=openbao

Signed-off-by: Alexander Scheel <[email protected]>

* Add changelog entry

Signed-off-by: Alexander Scheel <[email protected]>

---------

Signed-off-by: Alexander Scheel <[email protected]>
Nerkho pushed a commit to Nerkho/openbao that referenced this pull request Feb 26, 2025
* Fully upgrade Docker dependency

This follows churn caused by Docker removing type aliases in use by
others.

See also: moby/moby#47148

Signed-off-by: Alexander Scheel <[email protected]>

* Remove support for Solaris builds

Solaris has no support in upstream Moby any more; while we currently
only test with Docker, we may wish to support containerized plugins
running on remote hosts in the future and would be unable to do so from
Solaris.

Additionally, this release target sees few downloads and thus isn't
worth maintainer time to support.

See also: moby/moby#35373
See also: https://tooomm.github.io/github-release-stats/?username=openbao&repository=openbao

Signed-off-by: Alexander Scheel <[email protected]>

* Add changelog entry

Signed-off-by: Alexander Scheel <[email protected]>

---------

Signed-off-by: Alexander Scheel <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants