On Sat Dec 17 10:41:32 2022 +0000, Etaash Mathamsetty wrote:
> I also think you should squash c4e27bd0483c0a1fb872111b6c03b9ad240d85da
> (mark tests as todo) and d02be967f9cca0805b5eeaef6a74b3ec51a67619 (add
> tests) into one commit
I can do so, but before semi stub implementation even more tests are failing. See description. Should i squash mark all tests as todo and remove the markers with semi stub commit? Thanks for your help.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/1825#note_19792
Revision 24b26f8bd6a49c855eec417a6585d44c9458e3ec changed FreeBSD
(and DragonFly) not to use /proc any longer. Hence we also do not
need the exe_link variable on those two platforms, either. Avoid
declaring it there.
(This avoids a compiler warning about an unused variable with GCC 12.)
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/1833
find_device_from_devnode was unnecessarily guarded by HAVE_SYS_INOTIFY_H,
alas its use in process_monitor_event was not, so linking failed. Simply
make find_device_from_devnode generally available.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/1831
On Thu Dec 15 20:00:54 2022 +0000, Etaash Mathamsetty wrote:
> you should mark them as todo in the code for the tests then
I can do so, but before semi stub implementation even more tests are failing. See description. Should i squash mark all tests as todo and remove the markers with semi stub commit? Thanks for your help.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/1825#note_19782