extern/hishtory
1
0
Fork 0
mirror of https://github.com/ddworken/hishtory.git synced 2025-02-27 16:01:32 +01:00
Code Issues Packages Projects Releases Wiki Activity

Make bash support lenient with empty history lines, which seems to happen for the first command or two of new installs

Browse Source
This commit is contained in:
David Dworken 2024-02-18 17:54:45 -08:00
parent ce64c25d54
commit 3751e45717
No known key found for this signature in database
1 changed files with 6 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
client/cmd/saveHistoryEntry.go
View File

@ -412,6 +412,9 @@ func buildHistoryEntry(ctx context.Context, args []string) (*data.HistoryEntry,
func extractCommandFromArg(ctx context.Context, shell, arg string, isPresave bool) (string, error) {
if shell == "bash" {
cmd, err := getLastCommand(arg)
if cmd == "" {
return "", nil
}
if err != nil {
return "", fmt.Errorf("failed to build history entry: %w", err)
}
@ -583,6 +586,9 @@ func parseCrossPlatformTime(data string) time.Time {
}
func getLastCommand(history string) (string, error) {
if history == "" {
return "", nil
}
split := strings.SplitN(strings.TrimSpace(history), " ", 2)
if len(split) <= 1 {
return "", fmt.Errorf("got unexpected bash history line: %#v, please open a bug at github.com/ddworken/hishtory", history)