Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(252)

Unified Diff: src/platform/update_engine/generate_delta_main.cc

Issue 492008: AU: Try delta updates first, then full updates (Closed)
Patch Set: use mkstemp Created 11 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « src/platform/update_engine/gen_coverage_html.sh ('k') | src/platform/update_engine/install_action.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/platform/update_engine/generate_delta_main.cc
diff --git a/src/platform/update_engine/generate_delta_main.cc b/src/platform/update_engine/generate_delta_main.cc
index 14af193a9eb2d1b451d19e091a578f127f08c021..b64533b72afe4b448dedbd71e0f3ecf3d4da6404 100644
--- a/src/platform/update_engine/generate_delta_main.cc
+++ b/src/platform/update_engine/generate_delta_main.cc
@@ -4,45 +4,80 @@
#include <sys/types.h>
#include <sys/stat.h>
-#include <dirent.h>
-#include <fcntl.h>
-#include <stdlib.h>
-#include <stdio.h>
+#include <errno.h>
#include <unistd.h>
-
-#include <algorithm>
+#include <set>
#include <string>
-#include <vector>
-#include <tr1/memory>
-
#include <glib.h>
-
#include "chromeos/obsolete_logging.h"
+#include "update_engine/delta_diff_generator.h"
#include "update_engine/subprocess.h"
#include "update_engine/update_metadata.pb.h"
-
-using std::sort;
-using std::string;
-using std::vector;
-using std::tr1::shared_ptr;
+#include "update_engine/utils.h"
// This file contains a simple program that takes an old path, a new path,
// and an output file as arguments and the path to an output file and
// generates a delta that can be sent to Chrome OS clients.
+using std::set;
+using std::string;
+
namespace chromeos_update_engine {
-int main(int argc, char** argv) {
+namespace {
+// These paths should never be delta diffed. They should always be transmitted
+// in full in the update.
+const char* kNonDiffPaths[] = {
+ "/boot/extlinux.conf"
+};
+
+void usage(const char* argv0) {
+ printf("usage: %s old_dir new_dir out_file\n", argv0);
+ exit(1);
+}
+
+bool IsDir(const char* path) {
+ struct stat stbuf;
+ TEST_AND_RETURN_FALSE_ERRNO(lstat(path, &stbuf) == 0);
+ return S_ISDIR(stbuf.st_mode);
+}
+
+int Main(int argc, char** argv) {
g_thread_init(NULL);
Subprocess::Init();
if (argc != 4) {
usage(argv[0]);
}
+ logging::InitLogging("",
+ logging::LOG_ONLY_TO_SYSTEM_DEBUG_LOG,
+ logging::DONT_LOCK_LOG_FILE,
+ logging::APPEND_TO_OLD_LOG_FILE);
const char* old_dir = argv[1];
const char* new_dir = argv[2];
if ((!IsDir(old_dir)) || (!IsDir(new_dir))) {
usage(argv[0]);
}
- // TODO(adlr): implement using DeltaDiffGenerator
+
+ set<string> non_diff_paths;
+ for (size_t i = 0; i < arraysize(kNonDiffPaths); i++)
+ non_diff_paths.insert(kNonDiffPaths[i]);
+
+ DeltaArchiveManifest* manifest =
+ DeltaDiffGenerator::EncodeMetadataToProtoBuffer(new_dir);
+ CHECK(manifest);
+ CHECK(DeltaDiffGenerator::EncodeDataToDeltaFile(manifest,
+ old_dir,
+ new_dir,
+ argv[3],
+ non_diff_paths,
+ ""));
return 0;
-}
+}
+
+} // namespace {}
+
+} // namespace chromeos_update_engine
+
+int main(int argc, char** argv) {
+ return chromeos_update_engine::Main(argc, argv);
+}
« no previous file with comments | « src/platform/update_engine/gen_coverage_html.sh ('k') | src/platform/update_engine/install_action.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698