彤彤 李 commented on a discussion: 
https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/1250#note_150798


I have reworked the series and force-pushed the updated version.

The new version uses the imported FreeBSD sources as the basis for the RTEMS 
`libcsupport/src` passwd/group implementation. The FreeBSD files under 
`contrib/` are kept unchanged, and the RTEMS-specific files-only changes are in 
the follow-up commits.

While testing the reworked version, I found and fixed one passwd parsing 
compatibility issue. The parser now keeps the previous RTEMS behaviour where a 
missing final shell field is treated as an empty string.

I also ran these checks locally:

* `./waf` for `aarch64/zynqmp_qemu`
* `rtems-test` for `psxpasswd01`, `pwdgrp01`, and `pwdgrp02` on `zynqmp_qemu`
* manual `qemu-system-aarch64` runs for the same three tests, all reaching `*** 
END OF TEST ... ***`
* strict compile checks for `getgrent.c` and `getpwent.c` with `-Wall -Wextra 
-Werror -Wmissing-prototypes -Wstrict-prototypes`
* `git diff --check origin/main..HEAD`, excluding the unchanged imported 
FreeBSD `contrib` files

The latest GitLab pipeline also passes.

-- 
View it on GitLab: 
https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/1250#note_150798
You're receiving this email because of your account on gitlab.rtems.org.


_______________________________________________
bugs mailing list
[email protected]
http://lists.rtems.org/mailman/listinfo/bugs

Reply via email to