Ticket #1184 (closed defect: fixed)

Opened 7 years ago

Last modified 7 years ago

Tests failure, cannot install paludis

Reported by: eeight Owned by:
Priority: Sometime Milestone:
Component: core/paludis Version: 0.64.2
Keywords: Cc:
Blocked By: Blocking:
Distribution: Gentoo

Description

Actually, only fs_merger test fails. The very failure is

fs_merger_TEST.cc:484: Failure
Value of: Timestamp::now().seconds() - (data->root_dir / "dodgy_file").stat().mtim().seconds() >= (60 * 60 * 24 * 365 * 3) - 1
  Actual: false
Expected: true

I've plugged some debug print inside the test and got the timestamps for some particular run: now = 1313772780, file = 1219168343

As far as I can tell this is happening because 3 years ago here (Moscow, Russia) we had summer time at GMT+3 and now it is at GMT+4. DST is not longer in use in Russia. Because of that, mtime calculation is one hour off. Not sure how this should be fixed. Maybe increase error margin from one second to one hour plus one second?

Change History

comment:1 Changed 7 years ago by ciaranm

Maybe we should export TZ=UTC before running the test.

I hate timezones.

comment:2 Changed 7 years ago by ciaranm

Could you give me a TZ setting that triggers this please?

comment:3 Changed 7 years ago by eeight

I have /etc/timezone set to Europe/Moscow and date shows the following:

eeight ~ % date
Mon Aug 22 22:19:59 MSK 2011
eeight ~ % date -d "3 years ago"
Fri Aug 22 23:20:09 MSD 2008

comment:4 Changed 7 years ago by ciaranm

  • Status changed from new to closed
  • Resolution set to fixed
Note: See TracTickets for help on using tickets.