Bug 225384

Summary: [update] update textproc/p5-xmltv fix tv_grab_zz_sdjson
Product: Ports & Packages Reporter: Talal Al Dik <tad>
Component: Individual Port(s)Assignee: Thomas Zander <riggs>
Status: Closed FIXED    
Severity: Affects Some People CC: riggs
Priority: --- Keywords: needs-qa, patch
Version: LatestFlags: riggs: maintainer-feedback+
Hardware: Any   
OS: Any   
Attachments:
Description Flags
diff -ruN ./textproc/p5-xmltv.old/ ./textproc/p5-xmltv/ none

Description Talal Al Dik 2018-01-22 18:41:24 UTC
Created attachment 189975 [details]
diff -ruN ./textproc/p5-xmltv.old/ ./textproc/p5-xmltv/

The latest update to 5.70 introduced a new major update with tv_grab_zz_sdjson.
Unfortunately tv_grab_zz_sdjson has a linuxism and is not compatible with FreeBSD because p5-DateTime does not now time_zone local.
This update changes datetime time_zone to UTC.
Comment 1 Thomas Zander freebsd_committer freebsd_triage 2018-02-10 11:14:27 UTC
Doesn't this patch make the grabber just blindly assume that the local time zone is UTC?

When you start xmltv with setting the TZ environment variable, like e.g.

env TZ=Antarctica/McMurdo <xmltv with your favourite parameters here>

the module should work properly.
Comment 2 Talal Al Dik 2018-02-10 16:30:24 UTC
Indeed, I didn't know about the TZ-env workaround.

Also, I noticed that v1.45+ p5-DateTime is able to determine the "local" timezone unlike before.  

#!/usr/local/bin/perl -w
use DateTime;
my $dt_start = DateTime->today(time_zone => 'local');
print $dt_start;  #works now


I guess we can close this ticket at this point.