-
Notifications
You must be signed in to change notification settings - Fork 8
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add "after at" to schedule events for a specific time #2
Comments
Any chance this could be coordinated with efforts towards http://www.tcl.tk/cgi-bin/tct/tip/302 ? |
For the interim, there is a pure-tcl module for tcllib to do roughly this task: |
@jima00 - Yes, I would imagine that we'd want to combine this functionality with 302. My only trepidation is that it changes the semantics of [after] for all the programs that are using something like the 'cron' module to get the desired functionality. [at] really does need to be affected by changes in the system clock. |
It seems like implementing "after at" and then giving some time for programs currently using "after" to schedule wallclock events to switch to "after at" would be a path to making TIP #302 implementable. So I think this should be submitted as a new TIP with cross-reference to TIP #302. |
I've provided artificial PR sebres/tcl#4, that fully implements this (together with TIP #302 and many, many other things. If I get a bit time, and TCT does not see absolute no-go for it (because I've several doubts about), I'll make decent rebase to fossil (with commit-history and review). |
That looks really impressive. I’ll have to have a sitdown and look it all over. Hopefully it will be possible to convince the TCT.
From: "Serg G. Brester" <[email protected]>
Reply-To: flightaware/Tcl-bounties <[email protected]>
Date: Thursday, June 29, 2017 at 2:56 PM
To: flightaware/Tcl-bounties <[email protected]>
Cc: Peter da Silva <[email protected]>, Comment <[email protected]>
Subject: Re: [flightaware/Tcl-bounties] Add "after at" to schedule events for a specific time (#2)
I've provided artificial PR sebres/tcl#4<sebres/tcl#4>, that fully implements this (together with TIP #302<www.tcl.tk/cgi-bin/tct/tip/302.html> and many, many other things.
If I get a bit time, and TCT does not see absolute no-go for it (because I've several doubts about), I'll make decent rebase to fossil (with commit-history and review).
@flightaware<https://github.com/flightaware> I would like to know your statement also.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub<#2 (comment)>, or mute the thread<https://github.com/notifications/unsubscribe-auth/AAF0OoYMRiQVS4gqCZtaNNmFQbLUR5ziks5sJAGDgaJpZM4K2afA>.
|
I've extended test-cases a bit, and have provided a performance comparison for above-mentioned branch, if interesting sebres/tcl#4 (comment) |
Reintegrated resp. back-ported as 8.6 based branch in sebres-8-6-event-perf-branch |
A patch has been submitted upstream:
https://core.tcl.tk/tcl/tktview/2953b1b26b58c887435b74923ccef1712ad4b614
The text was updated successfully, but these errors were encountered: