Skip to content

fix dyn std::ops::FnMut() cannot be sent between threads safely #20 - #21

Open
alopatindev wants to merge 1 commit into
lholden:masterfrom
alopatindev:fix-issue-20
Open

fix dyn std::ops::FnMut() cannot be sent between threads safely #20#21
alopatindev wants to merge 1 commit into
lholden:masterfrom
alopatindev:fix-issue-20

Conversation

@alopatindev

@alopatindev alopatindev commented Jun 2, 2020

Copy link
Copy Markdown

No description provided.

@rakshith-ravi

rakshith-ravi commented Aug 11, 2020 •
edited
Loading

Copy link
Copy Markdown

@lholden can we have this merged and published to crates.io please?

@bellackn

bellackn commented May 20, 2021

Copy link
Copy Markdown

Seconding that @alopatindev - awesome crate, but this PR would make it even more awesome, @lholden! :) Tested it and this fixes it in my case:

    thread::spawn(move || {
        loop {
            schedule.tick();
            thread::sleep(std::time::Duration::from_secs(60));
        }
    });

BlackDex added a commit to BlackDex/job_scheduler that referenced this pull request Jun 1, 2022
@BlackDex
Update dependency versioning and Readme
4c59f85 
- Updated the dependency versioning so that a later version of uuid is able to be used.
- Updated the documentation to point to this repo.
- Applied an upstream PR to this repo: lholden#21
@BlackDex

BlackDex commented Jan 11, 2023

Copy link
Copy Markdown

This PR is implemented in https://crates.io/crates/job_scheduler_ng (with some minor changes).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@alopatindev @rakshith-ravi @bellackn @BlackDex