Merge pull request #2317 from deantvv/fix-touch-parse-date

Fix touch parse date error

src/uu/touch/src/touch.rs

@@ -238,10 +238,14 @@ fn parse_date(str: &str) -> FileTime {
     // be any simple specification for what format this parameter allows and I'm
     // not about to implement GNU parse_datetime.
     // http://git.savannah.gnu.org/gitweb/?p=gnulib.git;a=blob_plain;f=lib/parse-datetime.y
-    match time::strptime(str, "%c") {
-        Ok(tm) => local_tm_to_filetime(to_local(tm)),
-        Err(e) => panic!("Unable to parse date\n{}", e),
+    let formats = vec!["%c", "%F"];
+    for f in formats {
+        if let Ok(tm) = time::strptime(str, f) {
+            return local_tm_to_filetime(to_local(tm));
+        }
     }
+    show_error!("Unable to parse date: {}\n", str);
+    process::exit(1);
 }
 
 fn parse_timestamp(s: &str) -> FileTime {

tests/by-util/test_touch.rs

@@ -354,6 +354,34 @@ fn test_touch_set_date() {
     assert_eq!(mtime, start_of_year);
 }
 
+#[test]
+fn test_touch_set_date2() {
+    let (at, mut ucmd) = at_and_ucmd!();
+    let file = "test_touch_set_date";
+
+    ucmd.args(&["-d", "2000-01-23", file])
+        .succeeds()
+        .no_stderr();
+
+    assert!(at.file_exists(file));
+
+    let start_of_year = str_to_filetime("%Y%m%d%H%M", "200001230000");
+    let (atime, mtime) = get_file_times(&at, file);
+    assert_eq!(atime, mtime);
+    assert_eq!(atime, start_of_year);
+    assert_eq!(mtime, start_of_year);
+}
+
+#[test]
+fn test_touch_set_date_wrong_format() {
+    let (_at, mut ucmd) = at_and_ucmd!();
+    let file = "test_touch_set_date_wrong_format";
+
+    ucmd.args(&["-d", "2005-43-21", file])
+        .fails()
+        .stderr_contains("Unable to parse date: 2005-43-21");
+}
+
 #[test]
 fn test_touch_mtime_dst_succeeds() {
     let (at, mut ucmd) = at_and_ucmd!();