[PR] newt/pkg: Save all pkg.yml settings [mynewt-newt]

["sjanc (via GitHub) via notifications" <no...@mynewt.apache.org>]

sjanc opened a new pull request, #660:
URL: https://github.com/apache/mynewt-newt/pull/660

   LocalPackage.Save() wrote only a fixed set of fields (name, type,
   description, author, homepage, deps and build flags). Every other
   setting, including conditional ones like pkg.deps.FOO or
   pkg.cflags.FOO, pkg.init or pkg.keywords, was silently dropped. This
   made "newt target copy" produce incomplete targets and "newt target
   set" or "newt target amend" strip settings from existing ones.
       
   Write all settings from the package configuration. The common ones keep
   their current position, the rest follow sorted by name. Settings pulled
   in with $import are written inline and the directive itself is dropped,
   so the resulting file is self-contained.
       
   Fixes: https://github.com/apache/mynewt-newt/issues/150
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@mynewt.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] newt/pkg: Save all pkg.yml settings [mynewt-newt]

["sjanc (via GitHub) via notifications" <no...@mynewt.apache.org>]

sjanc merged PR #660:
URL: https://github.com/apache/mynewt-newt/pull/660


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@mynewt.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org