search

BambooEngine / ibus-bamboo

Bộ gõ tiếng Việt cho Linux
GNU General Public License v3.0
1.23k stars 76 forks source link

Port ibus-bamboo to freebsd #504

Closed goatastronaut0212 closed 2 months ago

goatastronaut0212 commented 4 months ago

Code đã tốt hơn không còn lỗi cũng như có thêm 1 test pipeline cho FreeBSD. PR này cũng sẽ đóng issue https://github.com/BambooEngine/ibus-bamboo/issues/347

Mình đã xóa PREFIX, và code sudo trong Makefile đi nên giờ người dùng sẽ phải tự thêm PREFIX hoặc sudo vào câu lệnh make để nó thực hiện được. Đây là ví dụ:

Bạn nghĩ thế nào @luongthanhlam ?

luongthanhlam commented 4 months ago

@goatastronaut0212 PR https://github.com/BambooEngine/goibus/pull/1 đã được merge, bạn update lại import path nhé

goatastronaut0212 commented 4 months ago

@luongthanhlam xin lỗi phiền bạn merge thêm PR https://github.com/BambooEngine/goibus/pull/2 được không? Vì lý do code merge trước chủ yếu là để mình có thể build được bên repo của mình. Với code mới này thì nó sử lại đường dẫn đúng cho BambooEngine với go.mod. Mình sẽ update lại import path của ibus-bamboo bên mình sớm thôi.

luongthanhlam commented 4 months ago

@luongthanhlam xin lỗi phiền bạn merge thêm PR BambooEngine/goibus#2 được không? Vì lý do code merge trước chủ yếu là để mình có thể build được bên repo của mình. Với code mới này thì nó sử lại đường dẫn đúng cho BambooEngine với go.mod. Mình sẽ update lại import path của ibus-bamboo bên mình sớm thôi.

ok, merge rồi đó bạn

goatastronaut0212 commented 4 months ago

Có vẻ như mình không fix được khúc test của Github Actions rồi. Nhưng build thì nó vẫn hoạt động. Mình rảnh thì mình sẽ sửa nó sau vậy

goatastronaut0212 commented 4 months ago

@luongthanhlam Mình đã thêm FreeBSD vô github actions test và xóa cái test không cần thiết trong step. Không biết bạn thấy có ổn với việc đó không?

Mình nghĩ việc trong thư mục vendor không có file test cũng là 1 điều dễ hiểu, thay vì chúng ta phải có thêm 1 lần chạy test vào trong Makefile rồi thêm mấy cái test vào thư mục vendor/github.com/BambooEngine/bamboo-core/ nó không được ổn cho lắm.

goatastronaut0212 commented 3 months ago

Mình đã đem file test lại từ bamboo-core vào vendor. Có vẻ là nó khá là ổn rồi. Mình cũng thêm Github Actions cho FreeBSD bạn nghĩ sao? Mình test thì nhận thấy không còn lỗi chuột nữa, mình nghĩ là nó ổn rồi