Quoting nathan binkert <[email protected]>:
Generally I'm in favor of this, but I think two characters are too
few, and the abbreviations are almost arbitrary in some places.
It's great to be brief and remove clutter but it's bad to be
cryptic. I'd be in favor of short but meaningful tags like [PARAMS]
or [ISA] or even [SWITCH HEADER].
I feel similarly, though with experience, we would get used to it. I know
that linux does something like this. What does it do for things other than
CC and LN?
Well, diving into their Makefile, I found this comment
# Beautify output
# ---------------------------------------------------------------------------
#
# Normally, we echo the whole command before executing it. By making
# that echo $($(quiet)$(cmd)), we now have the possibility to set
# $(quiet) to choose other forms of output instead, e.g.
#
# quiet_cmd_cc_o_c = Compiling $(RELDIR)/$@
# cmd_cc_o_c = $(CC) $(c_flags) -c -o $@ $<
#
# If $(quiet) is empty, the whole command will be printed.
# If it is set to "quiet_", only the short version will be printed.
# If it is set to "silent_", nothing will be printed at all, since
# the variable $(silent_cmd_cc_o_c) doesn't exist.
#
# A simple variant is to prefix commands with $(Q) - that's useful
# for commands that shall be hidden in non-verbose mode.
#
# $(Q)ln $@ :<
#
# If KBUILD_VERBOSE equals 0 then the above command will be hidden.
# If KBUILD_VERBOSE equals 1 then the above command is displayed.
So I ran the grep below and got the output after it. There are a lot
of examples of longer tags like DOCPROC or BUILD just looking at the
first few lines. Also note they don't use []s, although I'm not
deathly opposed if we decide to.
gbl...@cake /usr/src/linux $ grep -R quiet_cmd *
Documentation/kbuild/makefiles.txt: quiet_cmd_<command> - what
shall be echoed
Documentation/kbuild/makefiles.txt: quiet_cmd_image = BUILD $@
Documentation/DocBook/Makefile:quiet_cmd_docproc = DOCPROC $@
Documentation/DocBook/Makefile:quiet_cmd_db2ps = PS $@
Documentation/DocBook/Makefile:quiet_cmd_db2pdf = PDF $@
Documentation/DocBook/Makefile:quiet_cmd_db2html = HTML $@
Documentation/DocBook/Makefile:quiet_cmd_db2man = MAN $@
Documentation/DocBook/Makefile:quiet_cmd_fig2eps = FIG2EPS $@
Documentation/DocBook/Makefile:quiet_cmd_fig2png = FIG2PNG $@
Kbuild:quiet_cmd_bounds = GEN $@
Kbuild:quiet_cmd_offsets = GEN $@
Kbuild:quiet_cmd_syscalls = CALL $<
Makefile:# quiet_cmd_cc_o_c = Compiling $(RELDIR)/$@
Makefile:quiet_cmd_vmlinux__ ?= LD $@
Makefile:quiet_cmd_vmlinux_version = GEN .version
Makefile:quiet_cmd_sysmap = SYSMAP
Makefile:quiet_cmd_kallsyms = KSYM $@
Makefile:quiet_cmd_vmlinux-modpost = LD $@
Makefile:quiet_cmd_tags = GEN $@
Makefile:quiet_cmd_rmdirs = $(if $(wildcard $(rm-dirs)),CLEAN
$(wildcard $(rm-dirs)))
Makefile:quiet_cmd_rmfiles = $(if $(wildcard $(rm-files)),CLEAN
$(wildcard $(rm-files)))
Makefile:quiet_cmd_depmod = DEPMOD $(KERNELRELEASE)
Makefile:quiet_cmd_as_o_S = AS $@
arch/x86/boot/compressed/Makefile:quiet_cmd_relocs = RELOCS $@
arch/x86/boot/compressed/Makefile:quiet_cmd_mkpiggy = MKPIGGY $@
arch/x86/boot/Makefile:quiet_cmd_cpustr = CPUSTR $@
arch/x86/boot/Makefile:quiet_cmd_image = BUILD $@
arch/x86/boot/Makefile:quiet_cmd_voffset = VOFFSET $@
arch/x86/boot/Makefile:quiet_cmd_zoffset = ZOFFSET $@
arch/x86/tools/Makefile:quiet_cmd_posttest = TEST $@
arch/x86/lib/Makefile:quiet_cmd_inat_tables = GEN $@
arch/x86/kernel/cpu/Makefile:quiet_cmd_mkcapflags = MKCAP $@
arch/x86/vdso/Makefile:quiet_cmd_vdsosym = VDSOSYM $@
arch/x86/vdso/Makefile:quiet_cmd_vdso32sym = VDSOSYM $@
arch/x86/vdso/Makefile:quiet_cmd_vdso = VDSO $@
arch/x86/vdso/Makefile:quiet_cmd_vdso_install = INSTALL $@
arch/avr32/boot/images/Makefile:quiet_cmd_uimage = UIMAGE $@
arch/avr32/boot/images/Makefile:quiet_cmd_sfdwarf = SFDWARF $@
arch/avr32/Makefile:quiet_cmd_listing = LST $@
arch/avr32/Makefile:quiet_cmd_disasm = DIS $@
arch/cris/boot/compressed/Makefile:quiet_cmd_image = BUILD $@
arch/microblaze/boot/Makefile:quiet_cmd_cp = CP $< $...@$2
arch/microblaze/boot/Makefile:quiet_cmd_strip = STRIP $@
arch/microblaze/boot/Makefile:quiet_cmd_uimage = UIMAGE [email protected]
arch/microblaze/boot/Makefile:quiet_cmd_dtc = DTC $@
arch/alpha/boot/Makefile:quiet_cmd_strip = STRIP $@
arch/alpha/boot/Makefile:quiet_cmd_objstrip = OBJSTRIP $@
arch/um/kernel/Makefile:quiet_cmd_quote1 = QUOTE $@
arch/um/kernel/Makefile:quiet_cmd_quote2 = QUOTE $@
arch/sparc/boot/Makefile:quiet_cmd_elftoaout = ELFTOAOUT $@
arch/sparc/boot/Makefile:quiet_cmd_piggy = PIGGY $@
arch/sparc/boot/Makefile:quiet_cmd_btfix = BTFIX $@
arch/sparc/boot/Makefile:quiet_cmd_sysmap = SYSMAP $(obj)/System.map
arch/sparc/boot/Makefile:quiet_cmd_image = LD $@
arch/sparc/boot/Makefile:quiet_cmd_strip = STRIP $@
arch/sparc/boot/Makefile:quiet_cmd_piggy = PIGGY $@
arch/sparc/boot/Makefile:quiet_cmd_strip = STRIP $@
arch/arm/boot/Makefile:quiet_cmd_uimage = UIMAGE $@
arch/powerpc/boot/Makefile:quiet_cmd_copy_zlib = COPY $@
arch/powerpc/boot/Makefile:quiet_cmd_copy_zlibheader = COPY $@
arch/powerpc/boot/Makefile:quiet_cmd_copy_zliblinuxheader = COPY $@
arch/powerpc/boot/Makefile:quiet_cmd_copy_libfdt = COPY $@
arch/powerpc/boot/Makefile:quiet_cmd_bootcc = BOOTCC $@
arch/powerpc/boot/Makefile:quiet_cmd_bootas = BOOTAS $@
arch/powerpc/boot/Makefile:quiet_cmd_bootar = BOOTAR $@
arch/powerpc/boot/Makefile:quiet_cmd_wrap = WRAP $@
arch/powerpc/boot/Makefile:quiet_cmd_mkdir = MKDIR
$(patsubst $(INSTALL_HDR_PATH)/%,%,$@)
arch/powerpc/boot/Makefile:quiet_cmd_install = INSTALL $(patsubst
$(DESTDIR)$(WRAPPER_OBJDIR)/%,%,$@)
arch/powerpc/boot/Makefile:quiet_cmd_install_dts = INSTALL
$(patsubst $(DESTDIR)$(WRAPPER_DTSDIR)/%,dts/%,$@)
arch/powerpc/boot/Makefile:quiet_cmd_install_exe = INSTALL
$(patsubst $(DESTDIR)$(WRAPPER_BINDIR)/%,%,$@)
arch/powerpc/boot/Makefile:quiet_cmd_install_wrapper = INSTALL
$(patsubst $(DESTDIR)$(WRAPPER_BINDIR)/%,%,$@)
arch/powerpc/Makefile:quiet_cmd_relocs_check = CALL $<
arch/powerpc/platforms/cell/spufs/Makefile:quiet_cmd_spu_cc = SPU_CC $@
arch/powerpc/platforms/cell/spufs/Makefile:quiet_cmd_spu_as = SPU_AS $@
arch/powerpc/platforms/cell/spufs/Makefile:quiet_cmd_spu_ld = SPU_LD $@
arch/powerpc/platforms/cell/spufs/Makefile:quiet_cmd_spu_objcopy = OBJCOPY $@
arch/powerpc/platforms/cell/spufs/Makefile:quiet_cmd_hexdump = HEXDUMP $@
arch/powerpc/kernel/Makefile:quiet_cmd_systbl_chk = CALL $<
arch/powerpc/kernel/Makefile:quiet_cmd_prom_init_check = CALL $<
arch/powerpc/kernel/vdso64/Makefile:quiet_cmd_vdso64ld = VDSO64L $@
arch/powerpc/kernel/vdso64/Makefile:quiet_cmd_vdso64as = VDSO64A $@
arch/powerpc/kernel/vdso64/Makefile:quiet_cmd_vdso_install = INSTALL $@
arch/powerpc/kernel/vdso32/Makefile:quiet_cmd_vdso32ld = VDSO32L $@
arch/powerpc/kernel/vdso32/Makefile:quiet_cmd_vdso32as = VDSO32A $@
arch/powerpc/kernel/vdso32/Makefile:quiet_cmd_vdso_install = INSTALL $@
arch/xtensa/boot/lib/Makefile:quiet_cmd_copy_zlib = COPY $@
arch/xtensa/kernel/Makefile:quiet_cmd__cpp_lds_S = LDS $@
arch/s390/boot/compressed/Makefile:quiet_cmd_sizes = GEN $@
arch/s390/kernel/vdso64/Makefile:quiet_cmd_vdso64ld = VDSO64L $@
arch/s390/kernel/vdso64/Makefile:quiet_cmd_vdso64as = VDSO64A $@
arch/s390/kernel/vdso64/Makefile:quiet_cmd_vdso_install = INSTALL $@
arch/s390/kernel/vdso32/Makefile:quiet_cmd_vdso32ld = VDSO32L $@
arch/s390/kernel/vdso32/Makefile:quiet_cmd_vdso32as = VDSO32A $@
arch/s390/kernel/vdso32/Makefile:quiet_cmd_vdso_install = INSTALL $@
arch/blackfin/boot/Makefile:quiet_cmd_uimage = UIMAGE $@
arch/blackfin/boot/Makefile:quiet_cmd_mk_bin_xip = BIN $@
arch/ia64/kvm/Makefile:quiet_cmd_offsets = GEN $@
arch/ia64/hp/sim/boot/Makefile:quiet_cmd_cptotop = LN $@
arch/ia64/kernel/Makefile:quiet_cmd_nr_irqs = GEN $@
arch/ia64/kernel/Makefile:quiet_cmd_pv_check_sed = PVCHK $@
arch/ia64/kernel/Makefile.gate:quiet_cmd_gate = GATE $@
arch/sh/boot/Makefile:quiet_cmd_uimage = UIMAGE $@
arch/sh/kernel/vsyscall/Makefile:quiet_cmd_syscall = SYSCALL $@
drivers/md/Makefile:quiet_cmd_unroll = UNROLL $@
drivers/md/Makefile:quiet_cmd_mktable = TABLE $@
drivers/gpu/drm/radeon/Makefile:quiet_cmd_mkregtable = MKREGTABLE $@
drivers/net/wan/Makefile:quiet_cmd_build_wanxlfw = BLD FW $@
drivers/zorro/Makefile:quiet_cmd_devlist = DEVLIST $@
drivers/char/Makefile:quiet_cmd_conmk = CONMK $@
drivers/eisa/Makefile:quiet_cmd_eisaid = GEN $@
drivers/video/logo/Makefile:quiet_cmd_logo = LOGO $@
firmware/Makefile:quiet_cmd_mkdir = MKDIR $(patsubst $(objtree)/%,%,$@)
firmware/Makefile:quiet_cmd_ihex = IHEX $@
firmware/Makefile:quiet_cmd_ihex2fw = IHEX2FW $@
firmware/Makefile:quiet_cmd_h16tofw = H16TOFW $@
firmware/Makefile:quiet_cmd_fwbin = MK_FW $@
kernel/Makefile:quiet_cmd_ikconfiggz = IKCFG $@
kernel/Makefile:quiet_cmd_timeconst = TIMEC $@
lib/Makefile:quiet_cmd_crc32 = GEN $@
scripts/Makefile.host:quiet_cmd_host-csingle = HOSTCC $@
scripts/Makefile.host:quiet_cmd_host-cmulti = HOSTLD $@
scripts/Makefile.host:quiet_cmd_host-cobjs = HOSTCC $@
scripts/Makefile.host:quiet_cmd_host-cxxmulti = HOSTLD $@
scripts/Makefile.host:quiet_cmd_host-cxxobjs = HOSTCXX $@
scripts/Makefile.host:quiet_cmd_host-cshobjs = HOSTCC -fPIC $@
scripts/Makefile.host:quiet_cmd_host-cshlib = HOSTLLD -shared $@
scripts/Makefile.modinst:quiet_cmd_modules_install = INSTALL $@
scripts/mod/Makefile:quiet_cmd_elfconfig = MKELF $@
scripts/Makefile.modpost:quiet_cmd_modpost = MODPOST $(words
$(filter-out vmlinux FORCE, $^)) modules
scripts/Makefile.modpost:quiet_cmd_kernel-mod = MODPOST $@
scripts/Makefile.modpost:quiet_cmd_cc_o_c = CC $@
scripts/Makefile.modpost:quiet_cmd_ld_ko_o = LD [M] $@
scripts/Makefile.headersinst:quiet_cmd_install = INSTALL $(printdir)
($(words $(all-files))\
scripts/Makefile.headersinst:quiet_cmd_remove = REMOVE $(unwanted)
scripts/Makefile.headersinst:quiet_cmd_check = CHECK $(printdir)
($(words $(all-files)) files)
scripts/dtc/Makefile:quiet_cmd_bison = BISON $@
scripts/dtc/Makefile:quiet_cmd_flex = FLEX $@
scripts/Makefile.build: quiet_cmd_force_checksrc = CHECK $<
scripts/Makefile.build: quiet_cmd_checksrc = CHECK $<
scripts/Makefile.build:quiet_cmd_cc_s_c = CC $(quiet_modtag) $@
scripts/Makefile.build:quiet_cmd_cc_i_c = CPP $(quiet_modtag) $@
scripts/Makefile.build:quiet_cmd_cc_symtypes_c = SYM $(quiet_modtag) $@
scripts/Makefile.build:quiet_cmd_cc_o_c = CC $(quiet_modtag) $@
scripts/Makefile.build:quiet_cmd_cc_lst_c = MKLST $@
scripts/Makefile.build:quiet_cmd_as_s_S = CPP $(quiet_modtag) $@
scripts/Makefile.build:quiet_cmd_as_o_S = AS $(quiet_modtag) $@
scripts/Makefile.build:quiet_cmd_cpp_lds_S = LDS $@
scripts/Makefile.build:quiet_cmd_link_o_target = LD $@
scripts/Makefile.build:quiet_cmd_link_l_target = AR $@
scripts/Makefile.build:quiet_cmd_link_multi-y = LD $@
scripts/Makefile.build:quiet_cmd_link_multi-m = LD [M] $@
scripts/Makefile.lib:quiet_cmd_shipped = SHIPPED $@
scripts/Makefile.lib:quiet_cmd_ld = LD $@
scripts/Makefile.lib:quiet_cmd_objcopy = OBJCOPY $@
scripts/Makefile.lib:quiet_cmd_gzip = GZIP $@
scripts/Makefile.lib:quiet_cmd_bzip2 = BZIP2 $@
scripts/Makefile.lib:quiet_cmd_lzma = LZMA $@
scripts/Makefile.lib:quiet_cmd_lzo = LZO $@
scripts/package/Makefile:quiet_cmd_builddeb = BUILDDEB
scripts/genksyms/Makefile:quiet_cmd_keywords.c = GPERF $@
scripts/genksyms/Makefile:quiet_cmd_lex.c = FLEX $@
scripts/genksyms/Makefile:quiet_cmd_parse.c = BISON $@
scripts/Makefile.fwinst:quiet_cmd_install = INSTALL $(subst $(srctree)/,,$@)
scripts/Makefile.clean:quiet_cmd_clean = CLEAN $(obj)
scripts/Makefile.clean:quiet_cmd_cleandir = CLEAN $(__clean-dirs)
security/selinux/Makefile:quiet_cmd_flask = GEN $(obj)/flask.h
$(obj)/av_permissions.h
usr/Makefile:quiet_cmd_initfs = GEN $@
_______________________________________________
m5-dev mailing list
[email protected]
http://m5sim.org/mailman/listinfo/m5-dev
