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

Change package name 'gophercloud' to other #33

Open
sykim-etri opened this issue Mar 29, 2024 · 1 comment
Open

Change package name 'gophercloud' to other #33

sykim-etri opened this issue Mar 29, 2024 · 1 comment
Assignees

Comments

@sykim-etri
Copy link
Member

sykim-etri commented Mar 29, 2024

현재 package 이름이 아래와 같이 gophercloud로 유지되어 있는 상황입니다. 이에 따라 ./openstack 내 소스코드에서 gophercloud를 참고하고 있습니다.

package gophercloud

참고로 ./openstack 내 소스코드 수정시 vim에서는 아래와 같이 자동으로 "github.com/cloud-barista/nhncloud-sdk-go"를 gophercloud로 alias해주기도 하는 것으로 보입니다.

diff --git a/openstack/client.go b/openstack/client.go
index f86758c..5605f7a 100644
--- a/openstack/client.go
+++ b/openstack/client.go
@@ -7,7 +7,7 @@ import (
 
        cblog "github.com/cloud-barista/cb-log"
 
-       "github.com/cloud-barista/nhncloud-sdk-go"
+       gophercloud "github.com/cloud-barista/nhncloud-sdk-go"
        tokens2 "github.com/cloud-barista/nhncloud-sdk-go/openstack/identity/v2/tokens"
        "github.com/cloud-barista/nhncloud-sdk-go/openstack/identity/v3/extensions/ec2tokens"
        "github.com/cloud-barista/nhncloud-sdk-go/openstack/identity/v3/extensions/oauth1"

패키지명 변경에 대한 고려는 필요가 없을지 의견 부탁드립니다.
@innodreamer @powerkimhub

@powerkimhub powerkimhub added enhancement New feature or request and removed enhancement New feature or request labels Mar 29, 2024
@innodreamer
Copy link
Member

@sykim-etri @powerkimhub NHN SDK에서 package들중 이름이 gophercloud인 pkg의 이름을 변경했을때 SDK나 드라이버에 전체적으로 큰 문제가 발생하지 않으면 향후 변경 작업을 진행하도록 하겠습니다.

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

No branches or pull requests

3 participants