Axe the broken fpio support for good
authorPanu Matilainen <Panu Matilainen pmatilai@redhat.com>
Fri, 8 Jul 2011 07:43:00 +0000 (10:43 +0300)
committerPanu Matilainen <Panu Matilainen pmatilai@redhat.com>
Fri, 8 Jul 2011 08:09:58 +0000 (11:09 +0300)
commit3208913bdb201ab791f6b2b589d3d2ae3c87a5ad
tree19c2e3c8427c8970fbd627801a5180f6f0246f24
parent4b7e6cb79f5073fc42ae98e153668e574ebe0388
Axe the broken fpio support for good

- The whole thing is extremely hacky and failing to properly take
  fundamental differences in eg fread() and read() semantics into
  account (the whole rpmio fd is confused over this: Fread() claims
  to be fread() clone but reads behave with read() semantics
  and we dont provide feof() equivalent etc), special cased all over
  the io code when vectors exist for this purpose etc (not that fpio
  is the only offender in that regard)... RIP. Nothing should miss
  this but if we'd ever want to add it back, it'd better be done
  right at least.
- This COULD silently break somebody's code if they rely on the broken
  special case semantics of fpio mode, but most likely rpm itself
  was the only thing using fpio (and relying on the behavior) and
  even then, mostly only to get a FILE stream pointer out of it.
  Now we'll just silently return a fdio descriptor, as we do for any
  unrecognized io mode... (another sigh)
rpmio/rpmio.c