Redirect some headers noticed by the compiler #64

Unito
SlavekB ha unito 1 commit da feat/header-redirections a master 4 anni fa
selk 4 anni fa ha commentato
Collaboratori
/usr/include/sys/poll.h:1:2: warning: #warning redirecting incorrect #include <sys/poll.h> to <poll.h> [-Wcpp]
    1 | #warning redirecting incorrect #include <sys/poll.h> to <poll.h>
/usr/include/sys/signal.h:1:2: warning: #warning redirecting incorrect #include <sys/signal.h> to <signal.h> [-Wcpp]
    1 | #warning redirecting incorrect #include <sys/signal.h> to <signal.h>

Note: <sys/signal.h> has been removed from the tdeio/tdeio/connection.cpp source - it was already declared.
``` /usr/include/sys/poll.h:1:2: warning: #warning redirecting incorrect #include <sys/poll.h> to <poll.h> [-Wcpp] 1 | #warning redirecting incorrect #include <sys/poll.h> to <poll.h> /usr/include/sys/signal.h:1:2: warning: #warning redirecting incorrect #include <sys/signal.h> to <signal.h> [-Wcpp] 1 | #warning redirecting incorrect #include <sys/signal.h> to <signal.h> Note: <sys/signal.h> has been removed from the tdeio/tdeio/connection.cpp source - it was already declared. ```
SlavekB 4 anni fa ha commentato
Proprietario

Here's an interesting situation with sys/poll.h × poll.h:

  • On Linux (glibc) poll.h contains #include <sys/poll.h>, so the real file is really in `sys/poll.h
  • On FreeBSD poll.h is symlink to sys/poll.h, so the real file is really in sys/poll.h.

The mentioned incorrect seems to be on many systems as a real true file location. That's why I'm a little hesitant about what's right.

In any case, the test on Debian and FreeBSD is without problems.

Here's an interesting situation with `sys/poll.h` × `poll.h`: * On Linux (glibc) `poll.h` contains `#include <sys/poll.h>`, so the real file is really in `sys/poll.h * On FreeBSD `poll.h` is symlink to `sys/poll.h`, so the real file is really in `sys/poll.h`. The mentioned *incorrect* seems to be on many systems as a real true file location. That's why I'm a little hesitant about what's right. In any case, the test on Debian and FreeBSD is without problems.
selk 4 anni fa ha commentato
Autore
Collaboratori
https://pubs.opengroup.org/onlinepubs/9699919799/basedefs/poll.h.html#tag_13_34
SlavekB 4 anni fa ha commentato
Proprietario

Please can you rebase to the current HEAD so that your GPG signature can be preserved when merged?

Please can you rebase to the current HEAD so that your GPG signature can be preserved when merged?
selk 4 anni fa ha commentato
Autore
Collaboratori

Done. 👍

Done. :+1:
SlavekB chiusa questa pull request 4 anni fa
SlavekB branch feat/header-redirections eliminato 4 anni fa
SlavekB aggiunta alle pietre miliari R14.0.8 release 4 anni fa
La pull request è stata unita come c2adce2e13.
Effettua l'accesso per partecipare alla conversazione.
Nessun revisore
Nessuna milestone
Nessuna assegnatario
2 Partecipanti
Notifiche
Data di scadenza

Nessuna data di scadenza impostata.

Dipendenze

Nessuna dipendenza impostata.

Riferimento: TDE/tdelibs#64
Caricamento…
Non ci sono ancora contenuti.