date: make "iso-strict" conforming for the UTC timezone
ISO 8601-1:2020-12 specifies that a zero timezone offset must be denoted with a "Z" suffix instead of the numeric "+00:00". Add the correponding special case to show_date() and a new test. Changing an established output format which might be depended on by scripts is always problematic, but here we choose to adhere more closely to the published standard. Reported-by: Michael Osipov <michael.osipov@innomotics.com> Signed-off-by: Beat Bolli <dev+git@drbeat.li> Signed-off-by: Junio C Hamano <gitster@pobox.com>maint
							parent
							
								
									945115026a
								
							
						
					
					
						commit
						69e2bee1a3
					
				
							
								
								
									
										14
									
								
								date.c
								
								
								
								
							
							
						
						
									
										14
									
								
								date.c
								
								
								
								
							|  | @ -342,14 +342,18 @@ const char *show_date(timestamp_t time, int tz, const struct date_mode *mode) | ||||||
| 				tm->tm_hour, tm->tm_min, tm->tm_sec, | 				tm->tm_hour, tm->tm_min, tm->tm_sec, | ||||||
| 				tz); | 				tz); | ||||||
| 	else if (mode->type == DATE_ISO8601_STRICT) { | 	else if (mode->type == DATE_ISO8601_STRICT) { | ||||||
| 		char sign = (tz >= 0) ? '+' : '-'; | 		strbuf_addf(&timebuf, "%04d-%02d-%02dT%02d:%02d:%02d", | ||||||
| 		tz = abs(tz); |  | ||||||
| 		strbuf_addf(&timebuf, "%04d-%02d-%02dT%02d:%02d:%02d%c%02d:%02d", |  | ||||||
| 				tm->tm_year + 1900, | 				tm->tm_year + 1900, | ||||||
| 				tm->tm_mon + 1, | 				tm->tm_mon + 1, | ||||||
| 				tm->tm_mday, | 				tm->tm_mday, | ||||||
| 				tm->tm_hour, tm->tm_min, tm->tm_sec, | 				tm->tm_hour, tm->tm_min, tm->tm_sec); | ||||||
| 				sign, tz / 100, tz % 100); | 		if (tz == 0) { | ||||||
|  | 			strbuf_addch(&timebuf, 'Z'); | ||||||
|  | 		} else { | ||||||
|  | 			strbuf_addch(&timebuf, tz >= 0 ? '+' : '-'); | ||||||
|  | 			tz = abs(tz); | ||||||
|  | 			strbuf_addf(&timebuf, "%02d:%02d", tz / 100, tz % 100); | ||||||
|  | 		} | ||||||
| 	} else if (mode->type == DATE_RFC2822) | 	} else if (mode->type == DATE_RFC2822) | ||||||
| 		strbuf_addf(&timebuf, "%.3s, %d %.3s %d %02d:%02d:%02d %+05d", | 		strbuf_addf(&timebuf, "%.3s, %d %.3s %d %02d:%02d:%02d %+05d", | ||||||
| 			weekday_names[tm->tm_wday], tm->tm_mday, | 			weekday_names[tm->tm_wday], tm->tm_mday, | ||||||
|  |  | ||||||
|  | @ -46,6 +46,7 @@ check_show () { | ||||||
| TIME='1466000000 +0200' | TIME='1466000000 +0200' | ||||||
| check_show iso8601 "$TIME" '2016-06-15 16:13:20 +0200' | check_show iso8601 "$TIME" '2016-06-15 16:13:20 +0200' | ||||||
| check_show iso8601-strict "$TIME" '2016-06-15T16:13:20+02:00' | check_show iso8601-strict "$TIME" '2016-06-15T16:13:20+02:00' | ||||||
|  | check_show iso8601-strict "$(echo "$TIME" | sed 's/+0200$/+0000/')" '2016-06-15T14:13:20Z' | ||||||
| check_show rfc2822 "$TIME" 'Wed, 15 Jun 2016 16:13:20 +0200' | check_show rfc2822 "$TIME" 'Wed, 15 Jun 2016 16:13:20 +0200' | ||||||
| check_show short "$TIME" '2016-06-15' | check_show short "$TIME" '2016-06-15' | ||||||
| check_show default "$TIME" 'Wed Jun 15 16:13:20 2016 +0200' | check_show default "$TIME" 'Wed Jun 15 16:13:20 2016 +0200' | ||||||
|  |  | ||||||
		Loading…
	
		Reference in New Issue
	
	 Beat Bolli
						Beat Bolli