{"action":"created","issue":{"url":"https://api.github.com/repos/mbedmicro/pyOCD/issues/411","repository_url":"https://api.github.com/repos/mbedmicro/pyOCD","labels_url":"https://api.github.com/repos/mbedmicro/pyOCD/issues/411/labels{/name}","comments_url":"https://api.github.com/repos/mbedmicro/pyOCD/issues/411/comments","events_url":"https://api.github.com/repos/mbedmicro/pyOCD/issues/411/events","html_url":"https://github.com/mbedmicro/pyOCD/issues/411","id":363506827,"node_id":"MDU6SXNzdWUzNjM1MDY4Mjc=","number":411,"title":"Please rename the repository and remove binary files","user":{"login":"codehelp","id":2697839,"node_id":"MDQ6VXNlcjI2OTc4Mzk=","avatar_url":"https://avatars3.githubusercontent.com/u/2697839?v=4","gravatar_id":"","url":"https://api.github.com/users/codehelp","html_url":"https://github.com/codehelp","followers_url":"https://api.github.com/users/codehelp/followers","following_url":"https://api.github.com/users/codehelp/following{/other_user}","gists_url":"https://api.github.com/users/codehelp/gists{/gist_id}","starred_url":"https://api.github.com/users/codehelp/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/codehelp/subscriptions","organizations_url":"https://api.github.com/users/codehelp/orgs","repos_url":"https://api.github.com/users/codehelp/repos","events_url":"https://api.github.com/users/codehelp/events{/privacy}","received_events_url":"https://api.github.com/users/codehelp/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":4,"created_at":"2018-09-25T10:19:39Z","updated_at":"2018-10-05T06:57:39Z","closed_at":null,"author_association":"NONE","body":"To be able to package pyocd for distributions, the tarball produced by `python3 setup.py sdist` needs significant mangling to remove the binary files which have no source and to rename the entire directory to lowercase only.\r\n\r\nPlease can the pyOCD directory be renamed to lowercase?\r\n\r\nPlease can the binaries and elf files not be included in the release?"},"comment":{"url":"https://api.github.com/repos/mbedmicro/pyOCD/issues/comments/427265232","html_url":"https://github.com/mbedmicro/pyOCD/issues/411#issuecomment-427265232","issue_url":"https://api.github.com/repos/mbedmicro/pyOCD/issues/411","id":427265232,"node_id":"MDEyOklzc3VlQ29tbWVudDQyNzI2NTIzMg==","user":{"login":"codehelp","id":2697839,"node_id":"MDQ6VXNlcjI2OTc4Mzk=","avatar_url":"https://avatars3.githubusercontent.com/u/2697839?v=4","gravatar_id":"","url":"https://api.github.com/users/codehelp","html_url":"https://github.com/codehelp","followers_url":"https://api.github.com/users/codehelp/followers","following_url":"https://api.github.com/users/codehelp/following{/other_user}","gists_url":"https://api.github.com/users/codehelp/gists{/gist_id}","starred_url":"https://api.github.com/users/codehelp/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/codehelp/subscriptions","organizations_url":"https://api.github.com/users/codehelp/orgs","repos_url":"https://api.github.com/users/codehelp/repos","events_url":"https://api.github.com/users/codehelp/events{/privacy}","received_events_url":"https://api.github.com/users/codehelp/received_events","type":"User","site_admin":false},"created_at":"2018-10-05T06:57:39Z","updated_at":"2018-10-05T06:57:39Z","author_association":"NONE","body":"The flash algos in `target/` like `target_CC3220SF.py` are fine as source code - these are target-specific references and addresses which only make sense as static data. The data is presented in a form which allows modification and that is what matters. These are included in the package. The files in `binaries/` is not included in the package, neither are the files in `elf_files`.\r\n\r\nI understand about the firmware binaries being for separate boards, the difference in context is familiar from issues I have with scripts destined to run on a DUT but packaged in LAVA.\r\n\r\nI've tried experimenting with simple changes to MANIFEST.in but setuptools insists on always including `binaries/` and `elf_files/` into the tarball coming from ``python3 setup.py sdist``. Whilst it is acceptable (for distribution & packaging) to remove these binaries from the tarball of the pyocd release, I do see that it would not make sense to remove these entirely or put these into a separate git repo for \"pyocd-data\" as that would be too much of a barrier for running the pyocd tests upstream. As it is, the pyocd tests won't be able to run on the packaged pyocd. That isn't a problem, it's just inevitable in how pyocd needs to operate. I can set the Debian CI to add the upstream git repo to the packaged code and then run the tests.\r\n\r\nSo, I'll repackage each released tarball to drop `binaries/` and `elf_files/` directories, that's an extra step in the packaging but is do-able. I'll drop the request to remove those directories from the released tarball.\r\n\r\nIt would still be nice to rename the `pyOCD` directory to `pyocd` at some point. I'll rename the description of this issue.\r\n"}}
{"actor":{"display_login":"codehelp"}}