Opened 7 years ago

Closed 7 years ago

#1484 closed enhancement (fixed)

Should csc-options be valid at the top level too?

Reported by: Mario Domenech Goulart Owned by:
Priority: not urgent at all Milestone: 5.0
Component: build system Version: 5.0.0
Keywords: csc-options Cc:
Estimated difficulty:

Description

Currently, csc-options is not valid at the top level of .egg files. That leads to a lot of code duplication in case eggs declare multiple components.

How about allowing a top level csc-options declaration that can be clobbered by component-specific csc-options declarations?

Change History (4)

comment:1 Changed 7 years ago by evhan

I think so, and link-options too.

comment:2 Changed 7 years ago by evhan

I think common options for all components should go in a (component-options ...) item (or something similarly named), rather than at the top level of the egg file. We should probably still limit it to csc-options and link-options, though.

comment:3 Changed 7 years ago by felix winkelmann

I would suggest to make it cumulative - local options get added to toplevel options.

Last edited 7 years ago by felix winkelmann (previous) (diff)

comment:4 Changed 7 years ago by Kooda

Resolution: → fixed
Status: new → closed

Fixed by 9b2b6bc717ed266f17721444e9ac4e6277aa0c01

Note: See TracTickets for help on using tickets.