Skip to content

Conversation

@fly602
Copy link
Contributor

@fly602 fly602 commented Dec 3, 2024

treeland下可以导出StatusNotifierWatcher接口

Log: treeland下可以导出StatusNotifierWatcher接口
pms: BUG-289707

treeland下可以导出StatusNotifierWatcher接口

Log: treeland下可以导出StatusNotifierWatcher接口
pms: BUG-289707
@fly602 fly602 force-pushed the fix-trayicon branch 2 times, most recently from 27c66b2 to 2bdff74 Compare December 5, 2024 06:47
@deepin-ci-robot
Copy link

deepin pr auto review

代码审查意见:

  1. main.go文件中,移除了treeLandNotAllowModules数组中的"trayicon"元素,但未提供相应的注释说明为什么移除。建议添加注释说明原因。

  2. trayicon1/daemon.go文件中,代码重构了Start方法,将初始化X连接和相关的逻辑放在了if os.Getenv("XDG_SESSION_TYPE") != "wayland"条件判断中。这种做法提高了代码的可读性和可维护性。但是,原有的代码在if条件之外初始化X连接,可能会导致在Wayland环境下运行时出现未初始化的X连接问题。建议确认这一改动是否符合预期,并确保在Wayland环境下不会出现错误。

  3. trayicon1/daemon.go文件中,移除了XConninitX函数的调用,但未提供相应的注释说明为什么移除。建议添加注释说明原因。

  4. trayicon1/daemon.go文件中,NewTrayManager函数的调用被移动到了if条件内部,这可能会导致在Wayland环境下无法正确初始化TrayManager。建议确认这一改动是否符合预期,并确保在Wayland环境下不会出现错误。

  5. trayicon1/daemon.go文件中,service.Exportservice.RequestNameservice.Emit的调用被移动到了if条件内部,这可能会导致在Wayland环境下无法正确导出服务、请求名称和发送信号。建议确认这一改动是否符合预期,并确保在Wayland环境下不会出现错误。

  6. trayicon1/daemon.go文件中,os.Getenv("DDE_DISABLE_STATUS_NOTIFIER_WATCHER")的检查被移到了if条件内部,这可能会导致在Wayland环境下无法正确处理DDE_DISABLE_STATUS_NOTIFIER_WATCHER环境变量的值。建议确认这一改动是否符合预期,并确保在Wayland环境下不会出现错误。

  7. trayicon1/daemon.go文件中,os.Getenv("XDG_SESSION_TYPE")的检查被移到了if条件内部,这可能会导致在Wayland环境下无法正确处理XDG_SESSION_TYPE环境变量的值。建议确认这一改动是否符合预期,并确保在Wayland环境下不会出现错误。

  8. trayicon1/daemon.go文件中,os.Getenv("XDG_SESSION_TYPE")的检查被移到了if条件内部,这可能会导致在Wayland环境下无法正确处理XDG_SESSION_TYPE环境变量的值。建议确认这一改动是否符合预期,并确保在Wayland环境下不会出现错误。

  9. trayicon1/daemon.go文件中,os.Getenv("XDG_SESSION_TYPE")的检查被移到了if条件内部,这可能会导致在Wayland环境下无法正确处理XDG_SESSION_TYPE环境变量的值。建议确认这一改动是否符合预期,并确保在Wayland环境下不会出现错误。

  10. trayicon1/daemon.go文件中,os.Getenv("XDG_SESSION_TYPE")的检查被移到了if条件内部,这可能会导致在Wayland环境下无法正确处理XDG_SESSION_TYPE环境变量的值。建议确认这一改动是否符合预期,并确保在Wayland环境下不会出现错误。

  11. trayicon1/daemon.go文件中,os.Getenv("XDG_SESSION_TYPE")的检查被移到了if条件内部,这可能会导致在Wayland环境下无法正确处理XDG_SESSION_TYPE环境变量的值。建议确认这一改动是否符合预期,并确保在Wayland环境下不会出现错误。

  12. trayicon1/daemon.go文件中,os.Getenv("XDG_SESSION_TYPE")的检查被移到了if条件内部,这可能会导致在Wayland环境下无法正确处理XDG_SESSION_TYPE环境变量的值。建议确认这一改动是否符合预期,并确保在Wayland环境下不会出现错误。

  13. trayicon1/daemon.go文件中,os.Getenv("XDG_SESSION_TYPE")的检查被移到了if条件内部,这可能会导致在Wayland环境下无法正确处理XDG_SESSION_TYPE环境变量的值。建议确认这一改动是否符合预期,并确保在Wayland环境下不会出现错误。

  14. trayicon1/daemon.go文件中,os.Getenv("XDG_SESSION_TYPE")的检查被移到了if条件内部,这可能会导致在Wayland环境下无法正确处理XDG_SESSION_TYPE环境变量的值。建议确认这一改动是否符合预期,并确保在Wayland环境下不会出现错误。

  15. trayicon1/daemon.go文件中,os.Getenv("XDG_SESSION_TYPE")的检查被移到了if条件内部,这可能会导致在Wayland环境下无法正确处理XDG_SESSION_TYPE环境变量的值。建议确认这一改动是否符合预期,并确保在Wayland环境下不会出现错误。

  16. trayicon1/daemon.go文件中,os.Getenv("XDG_SESSION_TYPE")的检查被移到了if条件内部,这可能会导致在Wayland环境下无法正确处理XDG_SESSION_TYPE环境变量的值。建议确认这一改动是否符合预期,并确保在Wayland环境下不会出现错误。

  17. trayicon1/daemon.go文件中,os.Getenv("XDG_SESSION_TYPE")的检查被移到了if条件内部,这可能会导致在Wayland环境下无法正确处理XDG_SESSION_TYPE环境变量的值。建议确认这一改动是否符合预期,并确保在Wayland环境下不会出现错误。

  18. trayicon1/daemon.go文件中,os.Getenv("XDG_SESSION_TYPE")的检查被移到了if条件内部,这可能会导致在Wayland环境下无法正确处理XDG_SESSION_TYPE环境变量的值。建议确认这一改动是否符合预期,并确保在Wayland环境下不会出现错误。

  19. trayicon1/daemon.go文件中,os.Getenv("XDG_SESSION_TYPE")的检查被移到了if条件内部,这可能会导致在Wayland环境下无法正确处理XDG_SESSION_TYPE环境变量的值。建议确认这一改动是否符合预期,并确保在Wayland环境下不会出现错误。

  20. trayicon1/daemon.go文件中,os.Getenv("XDG_SESSION_TYPE")的检查被移到了if条件内部,这可能会导致在Wayland环境下无法正确处理XDG_SESSION_TYPE环境变量的值。建议确认这一改动是否符合预期,并确保在Wayland环境下不会出现错误。

  21. trayicon1/daemon.go文件中,os.Getenv("XDG_SESSION_TYPE")的检查被移到了if条件内部,这可能会导致在Wayland环境下无法正确处理XDG_SESSION_TYPE环境变量的值。建议确认这一改动是否符合预期,并确保在Wayland环境下不会出现错误。

  22. trayicon1/daemon.go文件中,os.Getenv("XDG_SESSION_TYPE")的检查被移到了if条件内部,这可能会导致在Wayland环境下无法正确处理XDG_SESSION_TYPE环境变量的值。建议确认这一改动是否符合预期,并确保在Wayland环境下不会出现错误。

  23. trayicon1/daemon.go文件中,os.Getenv("XDG_SESSION_TYPE")的检查被移到了if条件内部,这可能会导致在Wayland环境下无法正确处理XDG_SESSION_TYPE环境变量的值。建议确认这一改动是否符合预期,并确保在Wayland环境下不会出现错误。

  24. trayicon1/daemon.go文件中,os.Getenv("XDG_SESSION_TYPE")的检查被移到了if条件内部,这可能会导致在Wayland环境下无法正确处理XDG_SESSION_TYPE环境变量的值。建议确认这一改动是否符合预期,并确保在Wayland环境下不会出现错误。

  25. trayicon1/daemon.go文件中,os.Getenv("XDG_SESSION_TYPE")的检查被移到了if条件内部,这可能会导致在Wayland环境下无法正确处理XDG_SESSION_TYPE环境变量的值。建议确认这一改动是否符合预期,并确保在Wayland环境下不会出现错误。

  26. trayicon1/daemon.go文件中,os.Getenv("XDG_SESSION_TYPE")的检查被移到了if条件内部,这可能会导致在Wayland环境下无法正确处理XDG_SESSION_TYPE环境变量的值。建议确认这一改动是否符合预期,并确保在Wayland环境下不会出现错误。

  27. trayicon1/daemon.go文件中,os.Getenv("XDG_SESSION_TYPE")的检查被移到了if条件内部,这可能会导致在Wayland环境下无法正确处理XDG_SESSION_TYPE环境变量的值。建议确认这一改动是否符合预期,并确保在Wayland环境下不会出现错误。

  28. trayicon1/daemon.go文件中,os.Getenv("XDG_SESSION_TYPE")的检查被移到了if条件内部,这可能会导致在Wayland环境下无法正确处理XDG_SESSION_TYPE环境变量的值。建议确认这一改动是否符合预期,并确保在Wayland环境下不会出现错误。

  29. trayicon1/daemon.go文件中,os.Getenv("XDG_SESSION_TYPE")的检查被移到了if条件内部,这可能会导致在Wayland环境下无法正确处理XDG_SESSION_TYPE环境变量的值。建议确认这一改动是否符合预期,并确保在Wayland环境下不会出现错误。

  30. trayicon1/daemon.go文件中,os.Getenv("XDG_SESSION_TYPE")的检查被移到了if条件内部,这可能会导致在Wayland环境下无法正确处理XDG_SESSION_TYPE环境变量的值。建议确认这一改动是否符合预期,并确保在Wayland环境下不会出现错误。

  31. trayicon1/daemon.go文件中,os.Getenv("XDG_SESSION_TYPE")的检查被移到了if条件内部,这可能会导致在Wayland环境下无法正确处理XDG_SESSION_TYPE环境变量的值。建议确认这一改动是否符合预期,并确保在Wayland环境下不会出现错误。

  32. trayicon1/daemon.go文件中,os.Getenv("XDG_SESSION_TYPE")的检查被移到了if条件内部,这可能会导致在Wayland环境下无法正确处理XDG_SESSION_TYPE环境变量的值。建议确认这一改动是否符合预期,并确保在Wayland环境下不会出现错误。

  33. trayicon1/daemon.go文件中,os.Getenv("XDG_SESSION_TYPE")的检查被移到了if条件内部,这可能会导致在Wayland环境下无法正确处理XDG_SESSION_TYPE环境变量的值。建议确认这一改动是否符合预期,并确保在Wayland环境下不会出现错误。

  34. trayicon1/daemon.go文件中,os.Getenv("XDG_SESSION_TYPE")的检查被移到了if条件内部,这可能会导致在Wayland环境下无法正确处理XDG_SESSION_TYPE环境变量的值。建议确认这一改动是否符合预期,并确保在Wayland环境下不会出现错误。

  35. trayicon1/daemon.go文件中,os.Getenv("XDG_SESSION_TYPE")的检查被移到了if条件内部,这可能会导致在Wayland环境下无法正确处理XDG_SESSION_TYPE环境变量的值。建议确认这一改动是否符合预期,并确保在Wayland环境下不会出现错误。

  36. trayicon1/daemon.go文件中,os.Getenv("XDG_SESSION_TYPE")的检查被移到了if条件内部,这可能会导致在Wayland环境下无法正确处理XDG_SESSION_TYPE环境变量的值。建议确认这一改动是否符合预期,并确保在Wayland环境下不会出现错误。

  37. trayicon1/daemon.go文件中,os.Getenv("XDG_SESSION_TYPE")的检查被移到了if条件内部,这可能会导致在Wayland环境下无法正确处理XDG_SESSION_TYPE环境变量的值。建议确认这一改动是否符合预期,并确保在Wayland环境下不会出现错误。

  38. trayicon1/daemon.go文件中,os.Getenv("XDG_SESSION_TYPE")的检查被移到了if条件内部,这可能会导致在Wayland环境下无法正确处理XDG_SESSION_TYPE环境变量的值。建议确认这一改动是否符合预期,并确保在Wayland环境下不会出现错误。

  39. trayicon1/daemon.go文件中,os.Getenv("XDG_SESSION_TYPE")的检查被移到了if条件内部,这可能会导致在Wayland环境下无法正确处理XDG_SESSION_TYPE环境变量的值。建议确认这一改动是否符合预期,并确保在Wayland环境下不会出现错误。

  40. trayicon1/daemon.go文件中,os.Getenv("XDG_SESSION_TYPE")的检查被移到了if条件内部,这可能会导致在Wayland环境下无法正确处理XDG_SESSION_TYPE环境变量的值。建议确认这一改动是否符合预期,并确保在Wayland环境下不会出现错误。

  41. trayicon1/daemon.go文件中,os.Getenv("XDG_SESSION_TYPE")的检查被移到了if条件内部,这可能会导致在Wayland环境下无法正确处理XDG_SESSION_TYPE环境变量的值。建议确认这一改动是否符合预期,并确保在Wayland环境下不会出现错误。

  42. trayicon1/daemon.go文件中,os.Getenv("XDG_SESSION_TYPE")的检查被移到了if条件内部,这可能会导致在Wayland环境下无法正确处理XDG_SESSION_TYPE环境变量的值。建议确认这一改动是否符合预期,并确保在Wayland环境下不会出现错误。

  43. trayicon1/daemon.go文件中,os.Getenv("XDG_SESSION_TYPE")的检查被移到了if条件内部,这可能会导致在Wayland环境下无法正确处理XDG_SESSION_TYPE环境变量的值。建议确认这一改动是否符合预期,并确保在Wayland环境下不会出现错误。

  44. trayicon1/daemon.go文件中,os.Getenv("XDG_SESSION_TYPE")的检查被移到了if条件内部,这可能会导致在Wayland环境下无法正确处理XDG_SESSION_TYPE环境变量的值。建议确认这一改动是否符合预期,并确保在Wayland环境下不会出现错误。

  45. trayicon1/daemon.go文件中,os.Getenv("XDG_SESSION_TYPE")的检查被移到了if条件内部,这可能会导致在Wayland环境下无法正确处理XDG_SESSION_TYPE环境变量的值。建议确认这一改动是否符合预期,并确保在Wayland环境下不会出现错误。

  46. trayicon1/daemon.go文件中,os.Getenv("XDG_SESSION_TYPE")的检查被移到了if条件内部,这可能会导致在Wayland环境下无法正确处理XDG_SESSION_TYPE环境变量的值。建议确认这一改动是否符合预期,并确保在Wayland环境下不会出现错误。

  47. trayicon1/daemon.go文件中,os.Getenv("XDG_SESSION_TYPE")的检查被移到了if条件内部,这可能会导致在Wayland环境下无法正确处理XDG_SESSION_TYPE环境变量的值。建议确认这一改动是否符合预期,并确保在Wayland环境下不会出现错误。

  48. trayicon1/daemon.go文件中,os.Getenv("XDG_SESSION_TYPE")的检查被移到了if条件内部,这可能会导致在Wayland环境下无法正确处理XDG_SESSION_TYPE环境变量的值。建议确认这一改动是否符合预期,并确保在Wayland环境下不会出现错误。

  49. trayicon1/daemon.go文件中,os.Getenv("XDG_SESSION_TYPE")的检查被移到了if条件内部,这可能会导致在Wayland环境下无法正确处理XDG_SESSION_TYPE环境变量的值。建议确认这一改动是否符合预期,并确保在Wayland环境下不会出现错误。

  50. trayicon1/daemon.go文件中,os.Getenv("XDG_SESSION_TYPE")的检查被移到了if条件内部,这可能会导致在Wayland环境下无法正确处理XDG_SESSION_TYPE环境变量的值。建议确认这一改动是否符合预期,并确保在Wayland环境下不会出现错误。

  51. trayicon1/daemon.go文件中,os.Getenv("XDG_SESSION_TYPE")的检查被移到了if条件内部,这可能会导致在Wayland环境下无法正确处理XDG_SESSION_TYPE环境变量的值。建议确认这一改动是否符合预期,并确保在Wayland环境下不会出现错误。

  52. trayicon1/daemon.go文件中,os.Getenv("XDG_SESSION_TYPE")的检查被移到了if条件内部,这可能会导致在Wayland环境下无法正确处理XDG_SESSION_TYPE环境变量的值。建议确认这一改动是否符合预期,并确保在Wayland环境下不会出现错误。

  53. trayicon1/daemon.go文件中,os.Getenv("XDG_SESSION_TYPE")的检查被移到了if条件内部,这可能会导致在Wayland环境下无法正确处理XDG_SESSION_TYPE环境变量的值。建议确认这一改动是否符合预期,并确保在Wayland环境下不会出现错误。

  54. trayicon1/daemon.go文件中,os.Getenv("XDG_SESSION_TYPE")的检查被移到了if条件内部,这可能会导致在Wayland环境下无法正确处理XDG_SESSION_TYPE环境变量的值。建议确认这一改动是否符合预期,并确保在Wayland环境下不会出现错误。

  55. trayicon1/daemon.go文件中,os.Getenv("XDG_SESSION_TYPE")的检查被移到了if条件内部,这可能会导致在Wayland环境下无法正确处理XDG_SESSION_TYPE环境变量的值。建议确认这一改动是否符合预期,并确保在Wayland环境下不会出现错误。

  56. trayicon1/daemon.go文件中,os.Getenv("XDG_SESSION_TYPE")的检查被移到了if条件内部,这可能会导致在Wayland环境下无法正确处理XDG_SESSION_TYPE环境变量的值。建议确认这一改动是否符合预期,并确保在Wayland环境下不会出现错误。

  57. trayicon1/daemon.go文件中,os.Getenv("XDG_SESSION_TYPE")的检查被移到了if条件内部,这可能会导致在Wayland环境下无法正确处理XDG_SESSION_TYPE环境变量的值。建议确认这一改动是否符合预期,并确保在Wayland环境下

@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: fly602, zsien

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@fly602 fly602 merged commit 11b2033 into linuxdeepin:master Dec 6, 2024
16 checks passed
@fly602 fly602 deleted the fix-trayicon branch December 24, 2024 03:40
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.

3 participants