Skip to content

Avoid using a bunch of cfg flags and use of hardcoded auto.conf path #22

Closed

Description

@kloenk

Currently, building with O=$something fails as auto.conf is search for in the source tree.
Also the arguments passed as cfg parsed from auto.conf can get bigger as the max argument length of linux.

Activity

  1. kloenk commented on Oct 10, 2020

    @kloenk
    MemberAuthor

    For configs, that are y, do we want to pass --cfg=name="y" or do we just want --cfg=name

  2. added this to the milestone on Oct 10, 2020
  3. ojeda commented on Oct 10, 2020

    @ojeda
    Member

    For configs, that are y, do we want to pass --cfg=name="y" or do we just want --cfg=name

    Good question. On the C side we have the IS_BUILTIN/MODULE/REACHABLE/ENABLED macros as well as just #ifdef etc.

    I guess for ease of use we could provide each CONFIG_X value for both the value and the "empty" value, i.e.:

    • For y: CONFIG_X and CONFIG_X="y".
    • For m: CONFIG_X and CONFIG_X="m".

    We can forget about the IS_REACHABLE for the moment (one can always put the equivalent predicate if needed).

    For other kinds of values (e.g. integers and strings), I don't think we want to set the CONFIG_X, but it does not hurt and would be consistent.

  4. added a commit that references this issue on Nov 28, 2020
    df6070b
  5. ojeda commented on Nov 28, 2020

    @ojeda
    Member

    For configs, that are y, do we want to pass --cfg=name="y" or do we just want --cfg=name

    Good question. On the C side we have the IS_BUILTIN/MODULE/REACHABLE/ENABLED macros as well as just #ifdef etc.

    I guess for ease of use we could provide each CONFIG_X value for both the value and the "empty" value, i.e.:

    • For y: CONFIG_X and CONFIG_X="y".
    • For m: CONFIG_X and CONFIG_X="m".

    We can forget about the IS_REACHABLE for the moment (one can always put the equivalent predicate if needed).

    For other kinds of values (e.g. integers and strings), I don't think we want to set the CONFIG_X, but it does not hurt and would be consistent.

    This is handled by #32

  6. added a commit that references this issue on Nov 28, 2020
    39484fa
  7. added a commit that references this issue on Nov 28, 2020
    c28ec6f
  8. 16 remaining items

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions