[bug#49281] Add dynaconf

[Sarah Morgensen]

Hello,

Thanks for the patches :) The fight against vendoring is eternal.

I have a few suggestions:

* Unvendoring or otherwise removing files from sources is typically done
  with a snippet in the origin rather than a patch, as it's much smaller
  and doesn't break when updating. It might look like (untested):

(origin
  ...
  (modules '((guix build utils)))
  (snippet
    '(begin
      ;; Remove vendored dependencies
      (let ((unvendor '("click" "dotenv" "ruamel" "toml")))
        (with-directory-excursion "dynaconf/vendor"
          (for-each delete-file-recursively unvendor))
        (with-directory-excursion "dynaconf/vendor_src"
          (for-each delete-file-recursively unvendor))))))

  You'll still have to have the edits to dynaconf as a patch, of course.

* You've still included a python-box package despite none of the
  packages in your patch using it.

* pep8-naming has released 12.0.0, and tests pass :)

* Some of your patches no longer apply on master, and you should rebase
  them before sending a revised patchset. Consider using the `--base`
  option with format-patch, which helps git know what commit the patch
  is based on when applying.

paul <goodoldpaul@autistici.org> writes:

> Hi Guixers :),
>
> I'm sending a patch series to add dynaconf.
>
> Thank you for your time,
>
> Giacomo

Best,

Sarah