Re: Deprecated current-milliseconds
Hi, On Thu, 7 Dec 2023 13:21:12 +0200 Lassi Kortela wrote: >> chicken time current-process-milliseconds > > Thanks. Here's a grep of current-milliseconds in > eggs-5-latest. Anybody got time to patch some? > > dbus/0.97/tests/run.scm > gochan/5.2.10/chibi-compat.scm > gochan/5.2.10/chicken-module5.scm > gochan/5.2.10/gochan-record-printer.scm > gochan/5.2.10/gochan.scm > gochan/5.2.10/readme.md > gochan/5.2.10/tests/fake-search.scm > gochan/5.2.10/tests/run.scm > gochan/5.2.10/tests/stress.scm > hopefully/0.2.6/tests/run.scm > http-session/2.10/http-session.scm > matchable/1.1/matchable-test.scm > matrico/0.5rel/matrico.scm > nrepl/5.0.8/readme.md > openssl/2.2.5/openssl.socket.scm > pigeon-hole/0.2.8/tests/run.scm > postgresql/4.1.4/postgresql.scm > r7rs/1.0.9/scheme.time.scm > salmonella-feeds/0.1.1/salmonella-feeds.scm > sendfile/1.11/sendfile.scm > sendfile/1.11/strategies/mmap.scm > sendfile/1.11/strategies/rw.scm > simple-timer/0.1.2/timeout.scm > socket/0.3.3/socket.scm > sqlite3pth/0.2.6/sqlite3pth.scm > srfi-18/0.1.7/srfi-18.scm > srfi-18/0.1.7/tests/mutex-test.scm > srfi-19/4.9.5/srfi-19-tm.scm > test/1.2/test-support.scm Please note that simply replacing current-milliseconds with current-process-milliseconds will break code in older CHICKEN versions (< 5.3.0). See the thread whose start is https://lists.nongnu.org/archive/html/chicken-users/2023-02/msg3.html for more information. If you want to get rid of the warning, use something like: (cond-expand ((or chicken-5.0 chicken-5.1 chicken-5.2) (define current-process-milliseconds current-milliseconds)) (else)) (taken from sendfile 2.0 -- the openssl egg has similar code.) All the best. Mario -- http://parenteses.org/mario
Re: Deprecated current-milliseconds
> On 7. Dec 2023, at 13:20, Peter Bex wrote: > > On Thu, Dec 07, 2023 at 12:48:34PM +0100, Vasilij Schneidermann wrote: >> Hello, >> >> the openssl egg does no longer produce the warning in version 2.2.5 by >> either importing the new or old identifier, depending on whichever is >> available. Feel free to use that approach in the other egg and consider that >> it may skew the grep results. > > So does the sendfile egg. > > Cheers, > Peter In the “matrico” egg (file “matrico.scm"), the string “current-milliseconds” just appears to provide “current-process-milliseconds” to CHICKEN version before 5.3, see: https://github.com/gramian/matrico/blob/main/matrico.scm#L66 . Best Christian
Re: Deprecated current-milliseconds
On Thu, Dec 07, 2023 at 12:48:34PM +0100, Vasilij Schneidermann wrote: > Hello, > > the openssl egg does no longer produce the warning in version 2.2.5 by either > importing the new or old identifier, depending on whichever is available. > Feel free to use that approach in the other egg and consider that it may skew > the grep results. So does the sendfile egg. Cheers, Peter signature.asc Description: PGP signature
Re: Deprecated current-milliseconds
Hello, the openssl egg does no longer produce the warning in version 2.2.5 by either importing the new or old identifier, depending on whichever is available. Feel free to use that approach in the other egg and consider that it may skew the grep results. Vasilij On December 7, 2023 12:21:12 PM GMT+01:00, Lassi Kortela wrote: >> chicken time current-process-milliseconds > >Thanks. Here's a grep of current-milliseconds in eggs-5-latest. Anybody got >time to patch some? > >dbus/0.97/tests/run.scm >gochan/5.2.10/chibi-compat.scm >gochan/5.2.10/chicken-module5.scm >gochan/5.2.10/gochan-record-printer.scm >gochan/5.2.10/gochan.scm >gochan/5.2.10/readme.md >gochan/5.2.10/tests/fake-search.scm >gochan/5.2.10/tests/run.scm >gochan/5.2.10/tests/stress.scm >hopefully/0.2.6/tests/run.scm >http-session/2.10/http-session.scm >matchable/1.1/matchable-test.scm >matrico/0.5rel/matrico.scm >nrepl/5.0.8/readme.md >openssl/2.2.5/openssl.socket.scm >pigeon-hole/0.2.8/tests/run.scm >postgresql/4.1.4/postgresql.scm >r7rs/1.0.9/scheme.time.scm >salmonella-feeds/0.1.1/salmonella-feeds.scm >sendfile/1.11/sendfile.scm >sendfile/1.11/strategies/mmap.scm >sendfile/1.11/strategies/rw.scm >simple-timer/0.1.2/timeout.scm >socket/0.3.3/socket.scm >sqlite3pth/0.2.6/sqlite3pth.scm >srfi-18/0.1.7/srfi-18.scm >srfi-18/0.1.7/tests/mutex-test.scm >srfi-19/4.9.5/srfi-19-tm.scm >test/1.2/test-support.scm >
Re: Deprecated current-milliseconds
chicken time current-process-milliseconds Thanks. Here's a grep of current-milliseconds in eggs-5-latest. Anybody got time to patch some? dbus/0.97/tests/run.scm gochan/5.2.10/chibi-compat.scm gochan/5.2.10/chicken-module5.scm gochan/5.2.10/gochan-record-printer.scm gochan/5.2.10/gochan.scm gochan/5.2.10/readme.md gochan/5.2.10/tests/fake-search.scm gochan/5.2.10/tests/run.scm gochan/5.2.10/tests/stress.scm hopefully/0.2.6/tests/run.scm http-session/2.10/http-session.scm matchable/1.1/matchable-test.scm matrico/0.5rel/matrico.scm nrepl/5.0.8/readme.md openssl/2.2.5/openssl.socket.scm pigeon-hole/0.2.8/tests/run.scm postgresql/4.1.4/postgresql.scm r7rs/1.0.9/scheme.time.scm salmonella-feeds/0.1.1/salmonella-feeds.scm sendfile/1.11/sendfile.scm sendfile/1.11/strategies/mmap.scm sendfile/1.11/strategies/rw.scm simple-timer/0.1.2/timeout.scm socket/0.3.3/socket.scm sqlite3pth/0.2.6/sqlite3pth.scm srfi-18/0.1.7/srfi-18.scm srfi-18/0.1.7/tests/mutex-test.scm srfi-19/4.9.5/srfi-19-tm.scm test/1.2/test-support.scm
Re: Deprecated current-milliseconds
> Would it be possible to mark deprecations with alternatives/documentation > to point users in the new direction? Yes, next time, such a warning should mention any replacements. There is also a DEPRECATED file in the sources that lists recent deprecations. Thanks for the suggestion. felix
Re: Deprecated current-milliseconds
I've seen this too, and I wish the deprecation warning mentioned this alternative. I think in this particular example, it's just a rename - bu I'm guessing. Would it be possible to mark deprecations with alternatives/documentation to point users in the new direction? K. On Wed, Dec 6, 2023, 23:12 Kon Lovett wrote: > > > > On Dec 6, 2023, at 1:58 PM, Lassi Kortela wrote: > > > > Hi, > > > > Version 5.3 csc gives the warning: > > > > Use of deprecated identifier `current-milliseconds' from module > `chicken.time'. > > > > for (at least) these modules: > > > > * scheme.time (r7rs) > > * sendfile > > * srfi-18 > > > > Is there a drop-in replacement for this procedure? > > chicken time current-process-milliseconds > > > > > -l > > > > >
Re: Deprecated current-milliseconds
> On Dec 6, 2023, at 1:58 PM, Lassi Kortela wrote: > > Hi, > > Version 5.3 csc gives the warning: > > Use of deprecated identifier `current-milliseconds' from module > `chicken.time'. > > for (at least) these modules: > > * scheme.time (r7rs) > * sendfile > * srfi-18 > > Is there a drop-in replacement for this procedure? chicken time current-process-milliseconds > > -l >
Deprecated current-milliseconds
Hi, Version 5.3 csc gives the warning: Use of deprecated identifier `current-milliseconds' from module `chicken.time'. for (at least) these modules: * scheme.time (r7rs) * sendfile * srfi-18 Is there a drop-in replacement for this procedure? -l