# HG changeset patch
# User ethanzonca
# Date 2009-07-25 18:30:12
# Node ID eb27dfe10342e56d6ebf346dd3b929612300f83d
# Parent  59546e9e38706ea27800581eb05c91e244974b26

Bugfixing, found some issues we need to address.

diff --git a/TODO b/TODO
--- a/TODO
+++ b/TODO
@@ -1,4 +1,4 @@
-The most challenging and important tasks are marked with an "*"
+The most challenging and important tasks are marked with an "*", some items marked with initials to show who needs to fix them.
 
 Master
 *-Add calls to remotio
@@ -6,6 +6,7 @@ Master
  -Write a stub for getting info from the tarball/validifying the tarball. Read distren-job.xml, a file in the tarball, to find out 1. which rendering system to use (that system, e.g. blender/povray, can read more specifics, such as name of file to pass to blender and frames. Options common between different systems will be handled in common as best as possible)
  -Write a stub for publishing file and constructing job description so that the job can be shared
 *-Other XML-writing/reading
+M-frame_finder() needs to return both the job number and the frame number
 
 Slave
  -Fix current bugs
@@ -19,4 +20,6 @@ Options
  -Rewrite some stuff, try to make it simpler
  -Move server and client confuse code into their individual files, rather than in the common file.
  -Review all confuse interfacing
- -Push patch for relative includes in confuse to the confuse developers
\ No newline at end of file
+ -Push patch for relative includes in confuse to the confuse developers
+ 
+ 
\ No newline at end of file
diff --git a/src/server/slave.c b/src/server/slave.c
--- a/src/server/slave.c
+++ b/src/server/slave.c
@@ -114,6 +114,7 @@ options_init(argc,argv,&my_cfg, myopts, 
 	}
 	if(slavestatus==2){
 		fprintf(stderr, "Finished frame %d in job %d",framenum,jobnum);
+		tell_the_server(DISTREN_FINISHED_ASSIGNMENT); // or implement this into the exec_X() functions, but that would be repetative
 		slavestatus=0;
 	}
 	if(slavestatus==0){
diff --git a/src/server/slavefuncs.c b/src/server/slavefuncs.c
--- a/src/server/slavefuncs.c
+++ b/src/server/slavefuncs.c
@@ -113,7 +113,6 @@ void exec_blender(struct blendjob* blend
   int ret;
   char *cmd[] = { "blender", "-b", "-o", output, input, "-f", frame_str, (char *)0 };
   ret = execv("/usr/bin/blender", cmd); // This path will be absolute for testing, should be relative to install on production
-  finish_frame(blendjob, frame); // Is this going to be client-side or server-side?
 }
 
 /* Update me when the time comes: