Restore stdin import for the hishtory import command

This commit is contained in:
David Dworken 2022-10-16 09:51:52 -07:00
parent d255131d68
commit ab7db995aa
2 changed files with 10 additions and 8 deletions

View File

@ -391,7 +391,7 @@ func CheckFatalError(err error) {
}
}
func ImportHistory(ctx *context.Context) (int, error) {
func ImportHistory(ctx *context.Context, shouldReadStdin bool) (int, error) {
config := hctx.GetConf(ctx)
if config.HaveCompletedInitialImport {
// Don't run an import if we already have run one. This avoids importing the same entry multiple times.
@ -407,11 +407,13 @@ func ImportHistory(ctx *context.Context) (int, error) {
return 0, fmt.Errorf("failed to parse zsh history: %v", err)
}
historyEntries = append(historyEntries, extraEntries...)
// extraEntries, err = readStdin()
// if err != nil {
// return 0, fmt.Errorf("failed to read stdin: %v", err)
// }
// historyEntries = append(historyEntries, extraEntries...)
if shouldReadStdin {
extraEntries, err = readStdin()
if err != nil {
return 0, fmt.Errorf("failed to read stdin: %v", err)
}
historyEntries = append(historyEntries, extraEntries...)
}
db := hctx.GetDb(ctx)
currentUser, err := user.Current()
if err != nil {

View File

@ -75,7 +75,7 @@ func main() {
if os.Getenv("HISHTORY_TEST") == "" {
fmt.Println("Importing existing shell history...")
ctx := hctx.MakeContext()
numImported, err := lib.ImportHistory(ctx)
numImported, err := lib.ImportHistory(ctx, false)
lib.CheckFatalError(err)
if numImported > 0 {
fmt.Printf("Imported %v history entries from your existing shell history\n", numImported)
@ -83,7 +83,7 @@ func main() {
}
case "import":
ctx := hctx.MakeContext()
numImported, err := lib.ImportHistory(ctx)
numImported, err := lib.ImportHistory(ctx, true)
lib.CheckFatalError(err)
if numImported > 0 {
fmt.Printf("Imported %v history entries from your existing shell history\n", numImported)