the problem explained
<Md> Manoj: can you check what is inside the batchfile innxmit is refusing?
<Manoj> md: the batchfile is a ls -1 listing of message files
<Md> Manoj: file names? With full path?
<Manoj> md: The message files are in /var/spool/suck/glaurung/
<Manoj> md: The message files are in /var/spool/suck/glaurung/
<Manoj> md: yup
<Md> Manoj: the only thing I can think is innxmit dropped support for
non-storage API batches
<Manoj> md: umm, that is not good, as it beraks things like suck
[...]
<Manoj> md: or more of a transition scheme: have suck say that the conflict
with the new inn2
<Manoj> md: we definitely need to have suck and inn have some kind of
conflicts so that innn just doens't stop working after an upgrade
<Md> Manoj: BTW, I think rpost is broken too when using the storage API
<Md> Manoj: yes, but I think this is a suck bug
<Manoj> md: I think we do not want to take inn2 into testing until *after* the
conflicts relations are in place.
<Manoj> md: how can it be a suck bug?
<Md> Manoj: it fails to use the storage API
<Md> Manoj: suck should really make a rnews batch file, it's abusing innxmit
[...]
<Manoj> md: that is the documented behaviour of innxmit.
<Md> Manoj: OK, I agree that the documentation is obsolete
<Manoj> md: then it is inn that changed it's behaviour; it is up to inn to
manage the transition.
<Md> Manoj: this is silly. innxmit is *NOT* a way to inject articles inside
INN
<Md> Manoj: you should make a trivial script which takes the articles from the
suck spool, adds #! rnews nnn headers and pipes them to rnews
<Manoj> md: an undocumented change, or even a documented but sudden change in
a pculic API with no trasition processes or upgrade paths is, IMHO, a grave
bug.
<Md> Manoj: I'll say it again: other programs are NOT supposed to use innxmit.
they should use rpost
<Manoj> md: That may well be true now. But innxmit has been exposed as a
public interface (why document it in section 8 if it is an internal
script?)
<Manoj> md: if you want people to not use it in the future, fine, that is your
right. But set an upgrade path in place.
<Manoj> md: I think you may want to file a bug on suck to resolve this
<Md> Manoj: it's documented because you have to know what to put in newsfeeds
to write the correct batchfile
<Md> Manoj: I'll Cc the suck maintainer
[...]
<Md> Manoj: agreed. it's actually much simpler to fix the bug in suck making
it create rnews batches than to teach innxmit again about files and paths
<Manoj> md: ok. But the conflicts lines need to go into both packages
<Manoj> so someone partially upgrading from potato to woody is not
unnecesarily distressed.
--
ciao,
Marco
Reply to: