Skip to content

Commit

Permalink
fix - remove solana client
Browse files Browse the repository at this point in the history
  • Loading branch information
POPPIN-FUMI committed Nov 11, 2024
1 parent 89b1418 commit 2f8e760
Show file tree
Hide file tree
Showing 3 changed files with 12 additions and 11 deletions.
8 changes: 8 additions & 0 deletions .changeset/tiny-rocks-wonder.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,8 @@
---
'@epics-dao/solv': patch
---

fix - remove solana client

`solv i` had a dependency on the solana client, which is not needed for v2.x.x.
This change removes the dependency on the solana client.
13 changes: 3 additions & 10 deletions packages/solv/src/cli/update/update.ts
Original file line number Diff line number Diff line change
@@ -1,18 +1,11 @@
import { AGAVE_VALIDATOR, LEDGER_PATH } from '@/config/constants'
import { LEDGER_PATH } from '@/config/constants'
import { spawnSync } from 'child_process'
import installAgave from '../install/installAgave'
import getSolanaCLI from '@/config/getSolanaCLI'

export const updateVersion = async (version: string) => {
const solanaCLI = getSolanaCLI()
if (solanaCLI === AGAVE_VALIDATOR) {
installAgave(version)
return
}
const cmd = [
`sh -c "$(curl -sSfL https://release.solana.com/v${version}/install)"`,
]
spawnSync(cmd.join(' && '), { shell: true, stdio: 'inherit' })
installAgave(version)
return
}

export const monitorUpdate = async (
Expand Down
2 changes: 1 addition & 1 deletion packages/solv/src/config/config.ts
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,7 @@ export enum MAINNET_TYPES {

// ⚠️ Please DO NOT forget to turn this to false if restart is not needed
export const NODE_RESTART_REQUIRED_MAINNET = false
export const NODE_RESTART_REQUIRED_TESTNET = true
export const NODE_RESTART_REQUIRED_TESTNET = false

export type CONFIG_TYPE = {
ID: string
Expand Down

0 comments on commit 2f8e760

Please sign in to comment.