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

Use repo skywire-utilities #1126

Merged
merged 14 commits into from
Mar 9, 2022
Merged

Use repo skywire-utilities #1126

merged 14 commits into from
Mar 9, 2022

Conversation

ersonp
Copy link
Contributor

@ersonp ersonp commented Mar 4, 2022

Did you run make format && make check?

Fixes #1117

Changes:

  • Used packages from skywire-utilities instead of dmsg

How to test this PR:

  1. Checkout #dmsg-163
  2. Uncomment replace github.com/skycoin/dmsg => ../dmsg in go mod and run makedep; make build
  3. Start visor ./skywire-visor -c skywire-config.json
  4. Check if everything works

Needs:
#dmsg-163

ersonp added 8 commits March 4, 2022 20:12
This commit uses the package cipher from skywire-utilities instead of dmsg as it has been depriciated in it.
This commit uses the package buildinfo from skywire-utilities instead of dmsg as it has been depriciated in it.
This commit uses the package httputil from skywire-utilities instead of dmsg as it has been depriciated in it.
This commit uses the package httputil from skywire-utilities instead of dmsg and internal as they been depriciated.

The netutil package from internal has also been deleted.
This commit uses the package metricsutil from skywire-utilities instead of dmsg as it has been depriciated.
This commit uses the package cmdutil from skywire-utilities instead of dmsg as it has been depriciated.
This commit updates the makefile to use the buildinfo package from skywire-utilities instead of dmsg.
@ersonp ersonp marked this pull request as ready for review March 7, 2022 12:00
ersonp added 6 commits March 7, 2022 19:40
This commit uses the struct LocationData from the package geo from skywire-utilities and deletes the struct GeoLocation as it is replaced by LocationData and is no longer used..
@jdknives jdknives merged commit 4166f9b into skycoin:develop Mar 9, 2022
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