Skip to content
  • Mike Fleetwood's avatar
    Remove "../include/" from GParted header #includes · 8979913a
    Mike Fleetwood authored and Curtis Gedak's avatar Curtis Gedak committed
    It made the code look a little messy, is easily resolved in the build
    system and made the dependencies more complicated than needed.  Each
    GParted header was tracked via multiple different names (different
    numbers of "../include/" prefixes).  For example just looking at how
    DialogFeatures.o depends on Utils.h:
    
        $ cd src
        $ make DialogFeatures.o
        $ egrep ' [^ ]*Utils.h' .deps/DialogFeatures.Po
         ../include/DialogFeatures.h ../include/../include/Utils.h \
         ../include/../include/../include/../include/../include/../include/Utils.h \
         ../include/../include/../include/Utils.h \
    
    After removing "../include/" from the GParted header #includes, just
    need to add "-I../include" to the compile command via the AM_CPPFLAGS in
    src/Makefile.am.  Now the dependencies on GParted header files are
    tracked under a single name (with a single "../include/" prefix).  Now
    DialogFeatures.o only depends on a single name to Utils.h:
    
        $ make DialogFeatures.o
        $ egrep ' [^ ]*Utils.h' .deps/DialogFeatures.Po
         ../include/DialogFeatures.h ../include/Utils.h ../include/i18n.h \
    8979913a