Skip to content
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

Include target REQUIRE files in marshal #980

Closed
jasonatball opened this issue Apr 17, 2019 · 0 comments

Comments

Projects
None yet
2 participants
@jasonatball
Copy link
Collaborator

commented Apr 17, 2019

Describe the bug
The marshal code does not include files required by the target.txt REQUIRE keyword. This leads to old code being loaded when the server is restarted.

To Reproduce
Steps to reproduce the behavior:

  1. Require a conversion file in target.txt
  2. Launch the server
  3. Modify the conversion's initialize method
  4. Relaunch the server
  5. The old initialize code is loaded. However any new code in 'call' is run so it appears like all is well.

@jasonatball jasonatball self-assigned this Apr 17, 2019

@jasonatball jasonatball added the bug label Apr 17, 2019

ryanatball added a commit that referenced this issue Jun 27, 2019

@ryanatball ryanatball added this to the v4.4.0 milestone Jul 2, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.