Description of problem:

The --checkts option for createrepo only compares the timestamp on the repodata directory with the *first* rpm in the list of RPMs in the repo.

Version-Release number of selected component (if applicable):

createrepo-0.9.8-5.el6.noarch.rpm

How reproducible:

Almost all the time

Steps to Reproduce:
1. Use --checkts option

Additional info:

I tracked this down to a change in the source code between RHEL5 and RHEL6.

In RHEL5 (createrepo-0.4.11-3.el5.noarch.rpm) /usr/share/createrepo/genpkgmetadata.py:118

   118      def checkTimeStamps(self, directory):
   119          if self.cmds['checkts']:
   120              files = self.getFileList(self.cmds['basedir'], directory, '.
rpm')
   121              files = self.trimRpms(files)
   122              for f in files:
   123                  fn = os.path.join(self.cmds['basedir'], directory, f)
   124                  if not os.path.exists(fn):
   125                      errorprint(_('cannot get to file: %s') % fn)
   126                  if os.path.getctime(fn) > self.cmds['mdtimestamp']:
   127                      return False
   128          return True

In RHEL6 (createrepo-0.9.8-5.el6.noarch.rpm) /usr/lib/python2.6/site-packages/createrepo/__init__.py:288

   288      def checkTimeStamps(self):
   289          """check the timestamp of our target dir. If it is not newer tha
n 
   290             the repodata return False, else True"""
   291          if self.conf.checkts:
   292              dn = os.path.join(self.conf.basedir, self.conf.directory)
   293              files = self.getFileList(dn, '.rpm')
   294              files = self.trimRpms(files)
   295              for f in files:
   296                  fn = os.path.join(self.conf.basedir, self.conf.directory
, f)
   297                  if not os.path.exists(fn):
   298                      self.callback.errorlog(_('cannot get to file: %s') %
 fn)
   299                  if os.path.getctime(fn) > self.conf.mdtimestamp:
   300                      return False
   301                  else:
   302                      return True
   303                  
   304          return False

Lines 300 and 301 are not required.  They cause the function to return if the first RPM it looks at is older than the repodata.