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

Unified Diff: go/src/infra/tools/cr/cmd/firstrun/firstrun_windows.go

Issue 1929153002: Add beginnings of new cr command (Closed) Base URL: https://chromium.googlesource.com/infra/infra.git@master
Patch Set: Style and documentation Created 4 years, 8 months 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
Index: go/src/infra/tools/cr/cmd/firstrun/firstrun_windows.go
diff --git a/go/src/infra/tools/cr/cmd/firstrun/firstrun_windows.go b/go/src/infra/tools/cr/cmd/firstrun/firstrun_windows.go
new file mode 100644
index 0000000000000000000000000000000000000000..852f4db82e4b9732cc1fba1d5e6bcd5de8298093
--- /dev/null
+++ b/go/src/infra/tools/cr/cmd/firstrun/firstrun_windows.go
@@ -0,0 +1,49 @@
+// Copyright 2016 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+// Windows-specific function implementations for the firstrun subcommand.
+
+package firstrun
+
+import (
+ "bufio"
seanmccullough1 2016/04/29 00:54:36 None of these packages are used in this file. I gu
agable 2016/05/05 23:59:42 Right, removed.
+ "fmt"
+ "os"
+ "path"
+ "runtime"
+)
+
+// firstrunCheckNotInstalled is a sanity check to make sure that the user really
+// wants to do firstrun again, if it looks like cr is already installed.
+func firstrunCheckNotInstalled() {
+ // Check to see if a path ending in 'cr/bin' in in %PATH%.
+ // Check to see if an executable called 'cr.exe' is in that path.
+}
+
+// firstrunPromptInstallDir prompts the user for a directory path to install.
+func firstrunPromptInstallDir() {
+ // Figure out a sane place to suggest.
+ // Suggest it and ask the user for an alternative.
+ // Check that the chosen directory makes sense.
+}
+
+// firstrunInitInstallDir sets up the selected directory to house cr. It creates
+// the modules/ and bin/ subdirectories, places cr.exe in the top level, and
+// symlinks it into bin/.
+func firstrunInitInstallDir() {
+
+}
+
+// firstrunUpdatePath finds the registry entry for %PATH%, sees if it can
+// automatically update it, and prompts the user for permission to do so.
+func firstrunUpdatePath() {
+
+}
+
+// firstrunPrintUpdatePathInstructions prints instructions for the user to
+// update their %PATH% manually. This is used if updatePath fails, or if the
+// user declines to have their registry updated automatically.
+func firstrunPrintUpdatePathInstructions() {
+
+}

Powered by Google App Engine
This is Rietveld 408576698