Skip to content

fix: fix small bugs - #70

Open
dryfish09 wants to merge 1 commit into
VN-Linux-Family:mainfrom
dryfish09:fix/fix-purge-caramos-ota
Open

fix: fix small bugs#70
dryfish09 wants to merge 1 commit into
VN-Linux-Family:mainfrom
dryfish09:fix/fix-purge-caramos-ota

Conversation

@dryfish09

@dryfish09 dryfish09 commented Aug 17, 2026

Copy link
Copy Markdown

Vấn đề:

  • Không kiểm tra package tồn tại trước khi purge

    • dpkg -s có thể trả về 0 ngay cả khi package bị broken
    • Nên dùng dpkg-query -W hoặc apt list --installed
  • Xóa file .sources nhưng không update APT cache

    • Sau khi xóa /etc/apt/sources.list.d/caramos-ppa.sources, APT vẫn còn cache cũ.
    • Có thể gây lỗi khi chạy apt update sau này
  • -f caramos-ota-notifier quá rộng

  • Thiếu xóa file cấu hình user

  • Script chỉ xóa file system-wide

  • User config vẫn còn: ~/.config/caramos-ota/, ~/.local/share/caramos-ota/

  • Thiếu kiểm tra dịch vụ systemd còn chạy không

    • Sau khi disable, cần kiểm tra service đã stop hẳn chưa
  • Nếu có dữ liệu quan trọng trong /var/lib/caramos-ota, sẽ mất vĩnh viễn

Nên hỏi user hoặc backup

Cải thiện UX:

  • Thiếu --yes cho apt autoremove
  • Không xóa logrotate cũ

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.

1 participant