Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit ef80f0a1 authored by Geert Uytterhoeven's avatar Geert Uytterhoeven Committed by Michal Marek
Browse files

Documentation: kbuild: Improve if_changed documentation 



  - These days if_changed is used with many more commands than ld,
    objcopy, and gzip, hence add an ellipsis,
  - Any target that utilises if_changed must be listed in $(targets),
    so it needs an assignment to "targets", not "target".

Signed-off-by: default avatarGeert Uytterhoeven <geert@linux-m68k.org>
Signed-off-by: default avatarMichal Marek <mmarek@suse.cz>
parent b22ae40e

Documentation/kbuild/makefiles.txt

+3 −3
Original line number Diff line number Diff line
@@ -1092,7 +1092,7 @@ When kbuild executes, the following steps are followed (roughly): 


	Usage:

		target: source(s) FORCE

			$(call if_changed,ld/objcopy/gzip)

			$(call if_changed,ld/objcopy/gzip/...)



	When the rule is evaluated, it is checked to see if any files

	need an update, or the command line has changed since the last
@@ -1110,7 +1110,7 @@ When kbuild executes, the following steps are followed (roughly): 
	significant; for instance, the below will fail (note the extra space

	after the comma):

		target: source(s) FORCE

	#WRONG!#	$(call if_changed, ld/objcopy/gzip)

	#WRONG!#	$(call if_changed, ld/objcopy/gzip/...)



    ld

	Link target. Often, LDFLAGS_$@ is used to set specific options to ld.
@@ -1142,7 +1142,7 @@ When kbuild executes, the following steps are followed (roughly): 


	The ": %: %.o" part of the prerequisite is a shorthand that

	free us from listing the setup.o and bootsect.o files.

	Note: It is a common mistake to forget the "target :=" assignment,

	Note: It is a common mistake to forget the "targets :=" assignment,

	      resulting in the target file being recompiled for no

	      obvious reason.