broken, in the middle of edditting user / drv mod

install_full.sh

@@ -4,6 +4,7 @@ export nixstatepath=/nixstate/nix
 export ACLOCAL_PATH=/root/.nix-profile/share/aclocal
 
 if [ "$1" = "full" ]; then
+  nix-env-all-pkgs.sh -i gcc
   nix-env-all-pkgs.sh -i gnum4
   nix-env-all-pkgs.sh -i autoconf
   nix-env-all-pkgs.sh -i automake

src/libexpr/primops.cc

@@ -639,7 +639,7 @@ static Expr prim_derivationStrict(EvalState & state, const ATermVector & args)
 
     printMsg(lvlChatty, format("instantiated `%1%' -> `%2%'") % drvName % drvPath);
 
-	/* Write updated (no need to rebuild) state derivations to the database, so they can be updated at build time */	
+	/* Write updated (no need to rebuild) state derivations to the database, so they can be updated at build time	
 	if(enableState && !disableState){
 		if(store->isValidPath(outPath)){									//Only add when the path is already valid
 			Path deriver = queryDeriver(noTxn, outPath);					//query the deriver		
@@ -647,9 +647,8 @@ static Expr prim_derivationStrict(EvalState & state, const ATermVector & args)
 				store->addUpdatedStateDerivation(drvPath, outPath);						
 			}
 		}
-		//TODO Also add when path is not already valid, which drv does it take at build time, the latest ... guess so .. so we dont need to add?
 	}
-
+	*/
 
     /* Optimisation, but required in read-only mode! because in that
        case we don't actually write store expressions, so we can't

src/libstore/db.cc

@@ -203,9 +203,9 @@ void Database::open2(const string & path, bool removeOldEnv)
        run db_recover on the database to remove the existing DB
        environment (since changes only take effect on new
        environments). */
-    env->set_lk_max_locks(10000);
+    env->set_lk_max_locks(100000);
-    env->set_lk_max_lockers(10000);
+    env->set_lk_max_lockers(100000);
-    env->set_lk_max_objects(10000);
+    env->set_lk_max_objects(100000);
     env->set_lk_detect(DB_LOCK_DEFAULT);
     
     /* Dangerous, probably, but from the docs it *seems* that BDB

src/libstore/local-store.cc

@@ -82,11 +82,6 @@ static TableId dbDerivers = 0;
 */
 static TableId dbStateCounters = 0;
 
-/* Path
- 
-  */
-static TableId dbUpdatedDerivations = 0;
-
 
 bool Substitute::operator == (const Substitute & sub) const
 {
@@ -152,7 +147,6 @@ LocalStore::LocalStore(bool reserveSpace)
     dbSubstitutes = nixDB.openTable("substitutes");
     dbDerivers = nixDB.openTable("derivers");
     dbStateCounters = nixDB.openTable("statecounters");
-    dbUpdatedDerivations = nixDB.openTable("updatedDerivations");
 
     int curSchema = 0;
     Path schemaFN = nixDBPath + "/schema";
@@ -404,18 +398,42 @@ void LocalStore::queryReferrers(const Path & storePath,
 }
 
 
-void setDeriver(const Transaction & txn, const Path & storePath,
+void setDeriver(const Transaction & txn, const Path & storePath, const Path & deriver)
-    const Path & deriver)
 {
     assertStorePath(storePath);
     if (deriver == "") return;
     assertStorePath(deriver);
     if (!isRealisablePath(txn, storePath))
         throw Error(format("path `%1%' is not valid") % storePath);
+    
+    Derivation drv = derivationFromPath(deriver);		//Error if its a state component
+    if (drv.outputs.size() != 0)
+    	throw Error(format("path `%1%' is a state Path and its derivers need to be set by setDerivers") % storePath);
+    else
 	    nixDB.setString(txn, dbDerivers, storePath, deriver);
 }
 
 
+void setDerivers(const Transaction & txn, const Path & storePath, const PathSet & derivers)
+{
+	assertStorePath(storePath);
+    if (!isRealisablePath(txn, storePath))
+        throw Error(format("path `%1%' is not valid") % storePath);
+
+	Strings data;    	
+	for (PathSet::iterator i = derivers.begin(); i != derivers.end(); ++i){
+	   	string deriver = *i;
+	    assertStorePath(deriver);
+	   	Derivation drv = derivationFromPath(deriver);		
+    	if (drv.outputs.size() == 0)						//Error if its not a state component
+    		throw Error(format("path `%1%' is a NOT state Path at setDerivers") % storePath);
+       	data.push_back(deriver);
+	}
+	    	
+	nixDB.setStrings(txn, dbDerivers, storePath, data);					//update the db.
+}
+
+
 Path queryDeriver(const Transaction & txn, const Path & storePath)  
 {
     if (!isRealisablePath(txn, storePath))
@@ -428,6 +446,40 @@ Path queryDeriver(const Transaction & txn, const Path & storePath)
         return "";
 }
 
+//A '*' as argument stands for all identifiers or all users
+PathSet queryDerivers(const Transaction & txn, const Path & storePath, const string & identifier, const string & user)
+{
+	if (!isRealisablePath(txn, storePath))
+        throw Error(format("path `%1%' is not valid") % storePath);
+	
+	if(identifier == "" || user == "")
+		throw Error(format("The identifer or user argument is empty, use queryDeriver(...) for non-state components"));  
+	
+	Strings alldata;    	
+    nixDB.queryStrings(txn, dbDerivers, storePath, alldata);				//get all current derivers
+
+	PathSet filtereddata;
+	for (Strings::iterator i = alldata.begin(); i != alldata.end(); ++i) {
+		
+		string derivationpath = (*i);
+		Derivation drv = derivationFromPath(derivationpath);
+		
+		if (drv.outputs.size() != 1)
+			throw Error(format("The call queryDerivers with storepath %1% is not a statePath") % storePath);
+		
+		string getIdentifier = drv.stateOutputs.find("state")->second.stateIdentifier;
+		string getUser = drv.stateOutputs.find("state")->second.username;
+		
+		if( (getIdentifier == identifier || identifier == "*") && (getUser == user || user == "*") )
+			filtereddata.insert(derivationpath);
+	}
+	
+	if(filtereddata.size() == 0)
+		throw Error(format("There are no matching derivations with identifier %2% and user %3% for %1%") % storePath % identifier % user);
+	
+	return filtereddata;
+}
+
 
 const int substituteVersion = 2;
 
@@ -1170,30 +1222,6 @@ vector<int> LocalStore::getStatePathsInterval(const PathSet & statePaths)
     return nix::getStatePathsInterval(statePaths);
 }
 
-//TODO INCLUDE USERNAME + IDENTIFIER!!!!!
-Derivation getStateDerivation(const Path & path)
-{
-	Transaction txn(nixDB);			//TODO should u do a transaction here? ... this might delay the process ...
-	
-	string data;
-
-    //Get derivations of references    
-    nixDB.queryString(txn, dbDerivers, path, data);
-    //printMsg(lvlError, format("DERIVERS %1%") % data);
-    
-    txn.commit();
-    
-    Derivation drv = derivationFromPath(data);
-    if(drv.stateOutputs.size() == 0)
-   		throw Error(format("This path is not a state derivation: `%1%'") % path);
-    
-    return drv;
-}
-
-Derivation LocalStore::getStateDerivation(const Path & path)
-{
-    return nix::getStateDerivation(path);
-}
  
 //TODO direct or all recursive parameter
 //TODO check if these are state components
@@ -1224,51 +1252,51 @@ PathSet LocalStore::getStateReferencesClosure(const Path & path)
 }
 
 
-void addUpdatedStateDerivation(const Path & newdrv, const Path & storepath)
+//Merges a new .... into the list TODO
+//This is all about one store path
+//We assume newdrv is the newest
+PathSet mergeNewDerivationIntoList(const Path & storepath, const Path & newdrv, const PathSet drvs)
 {
-	Transaction txn(nixDB);
+	PathSet newdrvs;
 
-	//Check wheter were not duplicating an entry, if so, than we wont have to do anything
+	Derivation drv = derivationFromPath(newdrv);
-	//TODO
+	string identifier = drv.stateOutputs.find("state")->second.stateIdentifier;
-	if(false){
+	string user = drv.stateOutputs.find("state")->second.username;
 	
-		return;	
+	for (PathSet::iterator i = drvs.begin(); i != drvs.end(); ++i)	//Check if we need to remove old drvs
+    {
+    	Path drv = *i;
+    	Derivation getdrv = derivationFromPath(drv);
+    	string getIdentifier = getdrv.stateOutputs.find("state")->second.stateIdentifier;
+		string getUser = getdrv.stateOutputs.find("state")->second.username;
+		
+		if( !(identifier == getIdentifier && getUser == user) )		//only insert if it doenst already exist
+			newdrvs.insert(drv);
     }
     
-	Strings data;
+    newdrvs.insert(newdrv);
-	data.push_back(storepath);
+    return newdrvs;
-	time_t timestamp;
-  	time (&timestamp);
-  	string timestamp_s = time_t2string(timestamp);
-	printMsg(lvlError, format("Adding new drv (%1%) to replace drv of old component (%2%) with timestamp: %3%") % newdrv % storepath % timestamp_s);
-	data.push_back(timestamp_s);										//create a timestamp to remember which one was last inserted
-    nixDB.setStrings(txn, dbUpdatedDerivations, newdrv, data);
-    
-	txn.commit();		
-}
-
-void LocalStore::addUpdatedStateDerivation(const Path & newdrv, const Path & storepath)
-{
-	nix::addUpdatedStateDerivation(newdrv, storepath);
 }
 
 //TODO INCLUDE USERNAME + IDENTIFIER, the can be multiple derivations for the same component
+//TODO add mergeNewDerivationIntoList
 void updateAllStateDerivations()		  
 {
+	/*
 	Transaction txn(nixDB);
 
+	// Filter out all decrepated derivations, e.g. with a decrepated timestamp /	
 	Strings unique_paths;
-	
 	Strings keys;
     nixDB.enumTable(txn, dbUpdatedDerivations, keys);
     for (Strings::iterator i = keys.begin(); i != keys.end(); ++i)
     {
 		string drv_key = *i;	//the key is the derivation
     	Strings data;
+		
 		nixDB.queryStrings(txn, dbUpdatedDerivations, drv_key, data);
 	    string path = data.front();
     	bool exists = false;
-    	 
     	for (Strings::iterator j = unique_paths.begin(); j != unique_paths.end(); ++j)
     	{
 			string unique_path = *j;
@@ -1281,24 +1309,15 @@ void updateAllStateDerivations()
     		unique_paths.push_back(path);
     }
     
+    
+    
+	// Now we merge the two sets of: storepath -> [derivations] together into the db /
 	for (Strings::iterator i = unique_paths.begin(); i != unique_paths.end(); ++i)
     {
-    	string path = *i;
+    	string storepath = *i;
-    	//printMsg(lvlError, format("Unique: %1%") % path);
+    	printMsg(lvlError, format("Unique: %1%") % storepath);
-    	store->updateStateDerivation(txn, path);				//TODO replace store->
-    }
     	
-    txn.commit();
-}
     	
-void LocalStore::updateAllStateDerivations()
-{
-	nix::updateAllStateDerivations();
-}
-
-//TODO INCLUDE USERNAME + IDENTIFIER, the can be multiple derivations for the same component
-void updateStateDerivation(const Transaction & txn, const Path & storepath)		  
-{
     	Path drvPath = queryDeriver(txn, storepath);
 		Path originalDerivation = drvPath;
 		Path newDerivation = drvPath;					//the new drv path first equals the old one until a new one is found 
@@ -1346,13 +1365,21 @@ void updateStateDerivation(const Transaction & txn, const Path & storepath)
 			//TODO check wheter we update before or after ??
 			//TODO
 		}
+		
+    	
     }
     
-void LocalStore::updateStateDerivation(const Transaction & txn, const Path & storepath)
+    
-{
+    txn.commit();
-	nix::updateStateDerivation(txn, storepath);
+    */
 }
 
+void LocalStore::updateAllStateDerivations()
+{
+	nix::updateAllStateDerivations();
+}
+
+
 
 /* Upgrade from schema 1 (Nix <= 0.7) to schema 2 (Nix >= 0.8). */
 static void upgradeStore07()

src/libstore/local-store.hh

@@ -80,14 +80,10 @@ public:
 	
 	vector<int> getStatePathsInterval(const PathSet & statePaths);
 	
-	Derivation getStateDerivation(const Path & path);
-	
 	PathSet getStateReferencesClosure(const Path & path);
 	
 	void addUpdatedStateDerivation(const Path & newdrv, const Path & storepath);
 
-	void updateStateDerivation(const Transaction & txn, const Path & storepath);
-	
 	void updateAllStateDerivations();
 	
 };
@@ -155,6 +151,10 @@ void setDeriver(const Transaction & txn, const Path & path,
    deriver has been set. */
 Path queryDeriver(const Transaction & txn, const Path & path);
 
+/* Query the derivers of a state-store path. Gives an error (TODO?) if no
+   deriver has been set. */
+PathSet queryDerivers(const Transaction & txn, const Path & storePath, const string & identifier, const string & user);
+
 /* Delete a value from the nixStore directory. */
 void deleteFromStore(const Path & path, unsigned long long & bytesFreed);
 

src/libstore/remote-store.cc

@@ -388,13 +388,6 @@ vector<int> RemoteStore::getStatePathsInterval(const PathSet & statePaths)
     return intervals;
 }
 
-//TODO
-Derivation RemoteStore::getStateDerivation(const Path & path)
-{
-	Derivation d;
-	return d;
-}
-	
 //TODO
 PathSet RemoteStore::getStateReferencesClosure(const Path & path)
 {
@@ -402,18 +395,6 @@ PathSet RemoteStore::getStateReferencesClosure(const Path & path)
 	return empty;
 }
 
-//TODO
-void RemoteStore::addUpdatedStateDerivation(const Path & newdrv, const Path & storepath)
-{
-	
-}
-
-//TODO
-void RemoteStore::updateStateDerivation(const Transaction & txn, const Path & storepath)
-{
-	
-}
-
 //TODO
 void RemoteStore::updateAllStateDerivations()
 {

src/libstore/remote-store.hh

@@ -68,14 +68,8 @@ public:
 	
 	vector<int> getStatePathsInterval(const PathSet & statePaths);
 	
-	Derivation getStateDerivation(const Path & path);
-	
 	PathSet getStateReferencesClosure(const Path & path);
 
-	void addUpdatedStateDerivation(const Path & newdrv, const Path & storepath);
-
-	void updateStateDerivation(const Transaction & txn, const Path & storepath);
-	
 	void updateAllStateDerivations();
 	
     

src/libstore/store-api.hh

@@ -188,18 +188,9 @@ public:
 	/* TODO */
 	virtual vector<int> getStatePathsInterval(const PathSet & statePaths) = 0;
 	
-	/* TODO */	
-	virtual Derivation getStateDerivation(const Path & path) = 0;
-
 	/* TODO */	
 	virtual PathSet getStateReferencesClosure(const Path & path) = 0;
 	
-	/* TODO */	
-	virtual void addUpdatedStateDerivation(const Path & newdrv, const Path & storepath) = 0;
-
-	/* TODO */	
-	virtual void updateStateDerivation(const Transaction & txn, const Path & storepath) = 0;
-	
 	/* TODO */
 	virtual void updateAllStateDerivations() = 0;
 	

src/nix-state/nix-state.cc

@@ -57,18 +57,21 @@ Derivation getDerivation_andCheckArgs(Strings opFlags, Strings opArgs, Path & co
 		opArgs.pop_front();
 		stateIdentifier = opArgs.front();	
     }
-
 	string username = getCallingUserName();  
 
 	//printMsg(lvlError, format("%1% - %2% - %3% - %4% - %5%") % componentPath % statePath % stateIdentifier % binary % username);
     
+    PathSet drvs = queryDerivers(noTxn, componentPath, stateIdentifier, username);
+    if(drvs.size() != 1)
+    	throw UsageError("You must specify the full! binary path");
+    Derivation drv;
+    for (PathSet::iterator i = drvs.begin(); i != drvs.end(); ++i)		//ugly workaround for drvs[0].
+     	drv = derivationFromPath(*i);
      
-    //TODO check if this identifier exists !!!!!!!!!!!
+    queryDeriver(noTxn, "");
     
-    
-    Derivation drv = store->getStateDerivation(componentPath);
     DerivationStateOutputs stateOutputs = drv.stateOutputs; 
-    statePath = stateOutputs.find("state")->second.statepath;				//TODO STATEPATH BASED ON USERNAME AND INDENTIFIER !!!!!!!!!!!!!!!
+    statePath = stateOutputs.find("state")->second.statepath;
 	return drv;
 }
 
@@ -263,11 +266,19 @@ void run(Strings args)
 	store->addUpdatedStateDerivation("/nix/store/s6wggk924jx0gcb0l29ra4g9fxa3b4pp-hellohardcodedstateworld-1.0.drv", p2);		//
 	store->updateAllStateDerivations();
 	return;
+	
 	string a = makeStatePathFromGolbalHash("8f3b56a9a985fce54fd88c3e95a81a4b6b11fb98da12b977aee7f278c73ad3d7-hellohardcodedstateworld-1.0-test2", "kaaz");
 	printMsg(lvlError, format("%1%") % a);
 	return;
+	
 	printMsg(lvlError, format("Result: \"%1%\"") % getCallingUserName());
 	return;
+	*/
+	store = openStore();
+	//store->addUpdatedStateDerivation("/nix/store/bk4p7378ndm1p5qdr6a99wgxbiklilxy-hellohardcodedstateworld-1.0.drv", "/nix/store/vjijdazrn2jyzyk9sqwrl8fjq0qsmi8y-hellohardcodedstateworld-1.0");
+	
+	store->updateAllStateDerivations();
+	return;
 	
 	/* test */