Project

General

Profile

Actions

Bug #1576

open

wrong includes; mistakes <inlude> vs. "include"

Added by hd_brummy over 10 years ago. Updated over 10 years ago.

Status:
Feedback
Priority:
Normal
Assignee:
Start date:
10/27/2013
Due date:
% Done:

0%

Estimated time:

Description

bei mir compiliert der kram nicht,
es werden keine includes gefunden.

sourcen eigene includes sollte nur in den sourcen gesucht werden,
das wird erreicht wenn die includes in "includes" gesetzt werden
wenn die includes in <includes> eingeschlossen werden, werden diese irgendwo im systempath oder in /usr/include oder zusätlich im Makefile definierten pfaden gesucht,
anscheinend wird der tag INCLUDES += -I. (damit soll in den sourcen gesucht werden) von meinem gcc-4.8.1 nicht akzeptiert :(
ist syntaktich auch eher fragwürdig :D
bad example, #include <setup.h> kann zu fehlern führen, falls setup.h auch in irgenwelchen anderen pfaden gesucht und gefunden wird.

attached file will fix this issue

btw.

INCLUDES += -I$(VDRINCDIR) \
-I./imlibrenderer \
-I./imlibrenderer/fbrenderer \
-I./imlibrenderer/dmyrenderer

kann dann weg im Makefile, die "includes" sind jeweils vollpfadig getagt

#####
in dem patch sind auch 3 minor fixes fuer die po files enthalten

#####
weiterhin 2 fixes fuer das Makefile damits auf multicore cpu keine probleme gibt

make vs $(MAKE)


Files

Actions #1

Updated by hd_brummy over 10 years ago

oops,

mit ist noch ein include durch die finger gerutscht :(

imlibrenderer/fbrenderer/fbrenderer.c

-#include <fbrenderer.h>
+#include "fbrenderer.h"

Actions #2

Updated by horchi over 10 years ago

  • Status changed from New to Feedback

kann ich nicht nachvollziehen, mit dem 4.8.1 funktioniert es hier ohne jegliche Kompiler-Warnung,-Error oder ähnliches (4 Core System).

Die po Anpassungen und das $(MAKE) Makro übernehme ich gleich, Danke dafür!

Actions #3

Updated by hd_brummy over 10 years ago

horchi,

du hast recht
ich bin hier einem (bisher ubekannten bug) in meinen system aufgesessen
eine bloede kombination von -I$(VDRDIR)/include und der Makefile manipulation von meinem system im prepare teil
wie auch immer, das muss ich mir hier local nochmal genauer anschauen :/

die syntaktischen unkorrektheiten der includes

<include> vs. "include"

bestehen weiterhin!

Thanks

Actions

Also available in: Atom PDF