tsung icon indicating copy to clipboard operation
tsung copied to clipboard

replace deprecated `snmpm:sync_get/4` by `snmpm:sync_get2/4`

Open ariel-anieli opened this issue 1 year ago • 3 comments

snmpm:sync_get/4 is schedulled for removal in OTP 25.

ariel-anieli avatar Nov 19 '24 22:11 ariel-anieli

I am not sure if those changes are sufficient, gen_statem requires for example callback_mode() hook that i think is missing, and i think it also adds extra argument to state callback functions it calls, so i am not sure if this will be working as expected.

prefiks avatar Nov 20 '24 09:11 prefiks

I am not sure if those changes are sufficient, gen_statem requires for example callback_mode() hook that i think is missing, and i think it also adds extra argument to state callback functions it calls, so i am not sure if this will be working as expected.

Thanks for the feedback, @prefiks; I reduced the scope of the PR to snmpm:sync_get/4. I'll work on gen_statem in another PR: there is indeed a conversion guide from gen_fsm.

ariel-anieli avatar Nov 20 '24 22:11 ariel-anieli

Hello; any news on the PR?

ariel-anieli avatar Dec 02 '24 18:12 ariel-anieli