Repository navigation
fix: delay conky restart after network activation - #66
Merged
Merged
Conversation
実機Legacy BIOS環境・USB Wi-Fiアダプタ(wlx0090fee6a5f7)で、NetworkManager
dispatcherの"up"イベントが接続のactivated/default route確定より前に
発火し、Conky再起動時点で${gw_iface}が未確定のままDown/Upが0Bに固定
される不具合を修正した。
01-mypocketos-conky-restartで、既存Conkyプロセスをkillした後、
再起動直前に固定2秒のsleepを追加した。実機でWi-Fi切断→再接続を行い、
Down/Up表示が正常に更新されることを確認済み。
tests/desktop-polish/test_conky_network_restart.shに、sleepの存在・
配置順序(kill後・再起動前)の静的確認と、実際に約2秒の遅延が発生する
ことを計測するシナリオを追加した。
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
PC-FREEDOM
added a commit
that referenced
this pull request
Sep 23, 2026
Reflect the final pre-release hardware validation results in the spec. - UEFI + Secure Boot: on 2026-09-23, re-confirmed on the same physical UEFI machine, using the final candidate ISO (commit 41cbb23, which includes the Conky NetworkManager dispatcher fix) that the boot menu displays, the Standard Live desktop is reached, and Japanese display and basic UI have no major corruption (sections 3, 10.1.4, 16.2, 17) - Legacy BIOS, low-spec physical machine (Core 2 Duo T9550, 4GB RAM): confirmed Standard Live boot, USB Wi-Fi adapter recognition, NetworkManager connection, ping, and real Firefox communication (sections 16.2 and 17, new rows) - Persistence on a UEFI-created USB: confirmed on a physical UEFI machine that creating a Persistence area via the GUI, rebooting into "MyPocketOS Live (Persistence)", Conky showing Persistence mode, and persistence-test.txt surviving reboot (i.e. /home persistence) all work (sections 3, 16.2, 17) - Conky network display fix: added the technical background and the real-hardware confirmation that Down/Up now updates automatically after Wi-Fi connects, with no manual action, to section 5.3 - Minimum RAM: no new measurement was taken. The Calamares install requirement of "about 2 GiB or more" is kept, while the exact minimum RAM for the Live environment remains unmeasured; this is now explicitly treated as not a release-blocking gate (sections 16.2, 17) - Section 17 "GitHub CI PASS" row: replaced the stale HEAD reference (e5f5e16) with a statement that the two required status checks (mypocketos-tests, static-checks) both succeeded on 41cbb23 (PR #66, the last production-code change and the ISO build baseline), and that later doc-only PRs (e.g. #67) are not treated as the build baseline All entries in the section 17 completion table are now checked; none are left as an open box. Previously recorded confirmed dates and values are unchanged. The ISO build baseline commit (41cbb23) is unchanged. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Reproduction / verification context
wlx0090fee6a5f7)Down: 0B / Up: 0Bsleep 2before Conky restart and then reconnecting Wi-Fi made the display update correctlyTests
test_conky_network_restart.sh: 53/53git diff --check: cleanScope
reports/ai-review/changes