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

Update dependency modules; split repo into submodules #1238

Merged
merged 1 commit into from
Aug 10, 2018

Conversation

jrick
Copy link
Member

@jrick jrick commented Aug 10, 2018

This change updates dcrwallet to perform a module-aware build with the latest
versions of its dependencies. It also splits the repo from being a single
module to a collection of multiple modules for package consumption by other
projects.

All dcrwallet modules used by other dcrwallet modules use 'replace' directives
in their go.mod files to reference the dependency using a relative file path.
This allows the repo to have split published modules, and to refer to
unpublished module versions before tagging, while still allowing local
development and testing using the most recent changes of any given imported
major version of a module.

The following modules and versions are provided by this change:

github.com/decred/dcrwallet/[email protected]
github.com/decred/dcrwallet/[email protected]
github.com/decred/dcrwallet/[email protected]
github.com/decred/dcrwallet/internal/[email protected]
github.com/decred/dcrwallet/internal/[email protected]
github.com/decred/dcrwallet/[email protected]
github.com/decred/dcrwallet/[email protected]
github.com/decred/dcrwallet/[email protected]
github.com/decred/dcrwallet/[email protected]
github.com/decred/dcrwallet/[email protected]
github.com/decred/dcrwallet/ticketbuyer/[email protected]
github.com/decred/dcrwallet/[email protected]
github.com/decred/dcrwallet/[email protected]
github.com/decred/dcrwallet/[email protected]

A module in the project root also exists to provide reproducible builds, but
packages in this module are not intended for consumption as libraries for other
projects.

The repo must be tagged following this change to publish these modules and
versions to outside projects.

Copy link
Member

@davecgh davecgh left a comment

Choose a reason for hiding this comment

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

I have successfully built and tested all of the modules.

This change updates dcrwallet to perform a module-aware build with the latest
versions of its dependencies.  It also splits the repo from being a single
module to a collection of multiple modules for package consumption by other
projects.

All dcrwallet modules used by other dcrwallet modules use 'replace' directives
in their go.mod files to reference the dependency using a relative file path.
This allows the repo to have split published modules, and to refer to
unpublished module versions before tagging, while still allowing local
development and testing using the most recent changes of any given imported
major version of a module.

The following modules and versions are provided by this change:

  github.com/decred/dcrwallet/[email protected]
  github.com/decred/dcrwallet/[email protected]
  github.com/decred/dcrwallet/[email protected]
  github.com/decred/dcrwallet/internal/[email protected]
  github.com/decred/dcrwallet/internal/[email protected]
  github.com/decred/dcrwallet/[email protected]
  github.com/decred/dcrwallet/[email protected]
  github.com/decred/dcrwallet/[email protected]
  github.com/decred/dcrwallet/[email protected]
  github.com/decred/dcrwallet/[email protected]
  github.com/decred/dcrwallet/ticketbuyer/[email protected]
  github.com/decred/dcrwallet/[email protected]
  github.com/decred/dcrwallet/[email protected]
  github.com/decred/dcrwallet/[email protected]

A module in the project root also exists to provide reproducible builds, but
packages in this module are not intended for consumption as libraries for other
projects.

The repo must be tagged following this change to publish these modules and
versions to outside projects.
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.

2 participants