fix/makefile #9

simon.yhh · 2024-10-28T08:50:46Z

simon.yhh commented

2024-10-28 08:50:46 +00:00

問題

Makefile 執行 gen-rpc 時，會把 service.yaml 複製一份並改名為 service.example.yaml，不過 repo 內沒有 service.yaml 因此執行時會找不到檔案而出錯
服務啟動時會讀取 etc/notification.yaml ，但是這個檔案不會自動生成，除非手動新增否則會出錯
copy 指令在 Unix/Linux 系統中可能會找不到

改動

gen-rpc 改成 cp ./etc/service.example.yaml ./etc/notification.yaml

# 問題 1. Makefile 執行 `gen-rpc` 時，會把 `service.yaml` 複製一份並改名為 `service.example.yaml`，不過 repo 內沒有 `service.yaml` 因此執行時會找不到檔案而出錯 2. 服務啟動時會讀取 `etc/notification.yaml` ，但是這個檔案不會自動生成，除非手動新增否則會出錯 3. `copy` 指令在 Unix/Linux 系統中可能會找不到 # 改動 1. gen-rpc 改成 `cp ./etc/service.example.yaml ./etc/notification.yaml`

simon.yhh added 1 commit 2024-10-28 08:50:47 +00:00

c830647d0f fix: makefile copy config

simon.yhh self-assigned this 2024-10-28 08:50:55 +00:00

simon.yhh requested review from daniel.w 2024-10-28 08:51:04 +00:00

simon.yhh changed title from ~~WIP: fix/makefile~~ to fix/makefile

2024-10-28 08:51:07 +00:00

simon.yhh commented

2024-10-28 08:54:16 +00:00

備註：實測可以在本地端跑起來

daniel.w approved these changes 2024-10-29 05:30:12 +00:00

Makefile Outdated

													
				@ -23,3 +23,3 @@

				gen-rpc: # 建立 rpc code

					$(GO_CTL_NAME) rpc protoc ./generate/protobuf/notification.proto -m --style=$(GO_ZERO_STYLE) --go_out=./gen_result/pb --go-grpc_out=./gen_result/pb --zrpc_out=.

					copy ./etc/service.yaml ./etc/service.example.yaml

					cp ./etc/service.example.yaml ./etc/notification.yaml

daniel.w commented

2024-10-29 03:43:48 +00:00

這裡再 gen-rpc 原生產生整個資料夾時，時會自動生成 notification.yaml 在 etc 當中

但這邊牽扯到我不想直接把 notification.yaml 上傳到 git. 因為是設定的關係
所以我會想要手動再做一個 example 來只是說有這個東西，且格式怎麼寫，比較偏向不直接把他上傳

所以產生的時候才會想說把它變成
copy ./etc/service.yaml ./etc/service.example.yaml

這邊有一個更正
copy ./etc/notification.yaml ./etc/notification.example.yaml

另外當 config 有變時，example 檔案應該也同時新增或刪除才對

所以這裡的功用比較像是直接複製一個範例檔案，而不是起用的時後讓他可以有這個檔案

這裡再 gen-rpc 原生產生整個資料夾時，時會自動生成 notification.yaml 在 etc 當中但這邊牽扯到我不想直接把 notification.yaml 上傳到 git. 因為是設定的關係所以我會想要手動再做一個 example 來只是說有這個東西，且格式怎麼寫，比較偏向不直接把他上傳所以產生的時候才會想說把它變成 copy ./etc/service.yaml ./etc/service.example.yaml 這邊有一個更正 copy ./etc/notification.yaml ./etc/notification.example.yaml 另外當 config 有變時，example 檔案應該也同時新增或刪除才對所以這裡的功用比較像是直接複製一個範例檔案，而不是起用的時後讓他可以有這個檔案

simon.yhh commented

2024-10-29 09:02:24 +00:00

ok 那為什麼是用 copy 不是 cp ?

ok 那為什麼是用 `copy` 不是 `cp` ?

simon.yhh added 1 commit 2024-10-29 09:01:29 +00:00

89cc69154f fix: filename

This pull request can be merged automatically.

This branch is out-of-date with the base branch

You are not authorized to merge this pull request.

You can also view command line instructions.

Step 1:

From your project repository, check out a new branch and test the changes.

git checkout -b fix/makefile main

git pull origin fix/makefile

Step 2:

Merge the changes and update on Gitea.

git checkout main

git merge --no-ff fix/makefile

git push origin main

Sign in to join this conversation.

No reviewers

No Label

No Milestone

No project

No Assignees

2 Participants

Notifications

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: digimon/app-cloudep-notification-service#9