Re: Deprecated current-milliseconds

2023-12-07 Thread Christian Himpe



> 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

2023-12-07 Thread Peter Bex
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

2023-12-07 Thread Vasilij Schneidermann
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

2023-12-07 Thread Lassi Kortela

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

2023-12-07 Thread felix . winkelmann
> 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

2023-12-07 Thread Kristian Lein-Mathisen
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
> >
>
>
>