Skip to content

Commit

Permalink
targets.mk: fix inconsistent env variable BROKEN
Browse files Browse the repository at this point in the history
BROKEN TARGETS were built when BROKEN was set as a variable regardless of
it's value. Even an empty BROKEN variable caused BROKEN TARGETS to be
built. For DEVICES the number was evaluated and compared to true by Lua.

Make behavior consistent for relevant values UNSET/EMPTY/0 and 1:
UNSET/EMPTY/0: Don't build BROKEN TARGETS and DEVICES
1: Build BROKEN TARGETS and DEVICES

The behavior remains inconsistent for all positive numbers except 1.
(even floating point)
  • Loading branch information
Djfe committed Dec 12, 2023
1 parent 11a04bd commit bc8778c
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion targets/targets.mk
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ $(eval $(call GluonTarget,x86,legacy))
$(eval $(call GluonTarget,x86,64))


ifneq ($(BROKEN),)
ifeq ($(BROKEN),1)
$(eval $(call GluonTarget,bcm27xx,bcm2710)) # BROKEN: Untested
$(eval $(call GluonTarget,mvebu,cortexa9)) # BROKEN: No 11s support
endif

0 comments on commit bc8778c

Please sign in to comment.