Erstellt vor 8 Jahren

Zuletzt geändert vor 4 Jahren

#1092 assigned task

Revise makefiles, make them more modular, make some macros reusable in other contexts

Erstellt von: er13 Verantwortlicher: er13
Priorität: normal Meilenstein: freetz-future
Komponente: other Version: devel
Stichworte: Beobachter: er13
Product Id: Firmware Version:

Beschreibung

There are a couple of problems with our makefiles. They are not very modular, variable/macro definitions are spread all over the files (sometimes with no visible logic behind), include order is not very transparent, some code is duplicated as corresponding macros are too context specific. It would be worthwhile to eliminate all these deficiencies as it would reduce the maintenance costs.

The problems are however not that kind of problems one could not live with, so I'm setting 1.3 as the target milestone.

Änderungshistorie (21)

comment:1 Geändert vor 8 Jahren durch er13

(In [6216]) add TODO, refs #1092

comment:2 Geändert vor 7 Jahren durch oliver

Can you list/give some specific TODOs? Or do you plan to work on it?

comment:3 Geändert vor 6 Jahren durch cuma

  • Beobachter er13 hinzugefügt
  • Priorität von normal nach response-needed geändert

comment:4 Geändert vor 6 Jahren durch cuma

  • Meilenstein von freetz-1.3 nach freetz-future geändert

comment:5 Geändert vor 6 Jahren durch er13

In 9440:

build system:

  • introduce include/make dir and move some make-macros to it
  • eliminate some code clones (_comma,_space)
  • other minor changes like "export VARIABLE=value" instead of "VARIABLE=value; export VARIABLE"
  • refs #1092

comment:6 Geändert vor 4 Jahren durch er13

  • Meilenstein von freetz-future nach freetz-next geändert

comment:7 Geändert vor 4 Jahren durch er13

In 12590:

  • cleanup toolchain/make/download-toolchain.mk
  • refs #1092

comment:8 Geändert vor 4 Jahren durch er13

In 12591:

  • introduce & use a variable for UNPACK_TARBALL prerequisites
  • refs #1092

comment:9 Geändert vor 4 Jahren durch er13

In 12593:

makefiles under tools/make:

  • use UNPACK_TARBALL macro where possible
  • cleanup unpack/source recipes
  • refs #1092

comment:10 Geändert vor 4 Jahren durch er13

In 12600:

PKG_PATCH macro:

  • do not distinguish between PKG_BIN_PATCH & PKG_LIB_PATCH, since moving libs to dedicated subdirs this is not necessary anymore
  • refs #1092

comment:11 Geändert vor 4 Jahren durch er13

In 12592:

makefiles under toolchain/make:

  • use UNPACK_TARBALL macro
  • cleanup some unpack recipes
  • refs #1092

comment:12 Geändert vor 4 Jahren durch er13

In 12601:

  • tweak PKG_PATCH macro so that it can be used by host packages too
  • refs #1092

comment:13 Geändert vor 4 Jahren durch er13

In 12602:

  • use APPLY_PATCHES macro all over makefiles
  • refs #1092

comment:14 Geändert vor 4 Jahren durch er13

In 12653:

makefiles:

  • eliminate some code clones by introducing a corresponding macro (COPY_USING_TAR)
  • refs #1092

phpxmail:

  • cleanup makefile

comment:15 Geändert vor 4 Jahren durch er13

In 12673:

pkg-build-system:

  • eliminate code clone
  • move some code from PKG_UNPACKED_BIN__INT (wrong location as it has nothing to with unpacking) to PKG_FINISH_BIN__INT
  • add missing .PHONY's
  • refs #1092

comment:16 Geändert vor 4 Jahren durch er13

In 12674:

pkg-build-system:

  • eliminate $(PKG)_SOURCE_DIR related code clone
  • refs #1092

comment:17 Geändert vor 4 Jahren durch er13

In 12675:

pkg-build-system:

  • eliminate PKG_UNPACKED_BIN__INT vs. PKG_UNPACKED_LIB__INT code clone - both were identical since r12673
  • rename PKG_LOCALSOURCE_PACKAGE__INT to PKG_UNPACKED_LOCALSOURCE_PACKAGE__INT
  • refs #1092

comment:18 Geändert vor 4 Jahren durch er13

  • Status von new nach assigned geändert
  • Verantwortlicher auf er13 gesetzt

comment:19 Geändert vor 4 Jahren durch er13

  • Priorität von response-needed nach normal geändert

comment:20 Geändert vor 4 Jahren durch er13

In 12697:

pkg-build-system:

  • eliminate $(pkg)-dirclean related code clones
  • refs #1092

comment:21 Geändert vor 4 Jahren durch er13

  • Meilenstein von freetz-next nach freetz-future geändert
Hinweis: Hilfe zur Verwendung von Tickets finden Sie in TracTickets.