Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bump setuptools from 46.0.0 to 65.5.1 #63

Merged
merged 186 commits into from
Sep 13, 2023

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Dec 27, 2022

Bumps setuptools from 46.0.0 to 65.5.1.

Release notes

Sourced from setuptools's releases.

v65.5.1

No release notes provided.

v65.5.0

No release notes provided.

v65.4.1

No release notes provided.

v65.4.0

No release notes provided.

v65.3.0

No release notes provided.

v65.2.0

No release notes provided.

v65.1.1

No release notes provided.

v65.1.0

No release notes provided.

v65.0.2

No release notes provided.

v65.0.1

No release notes provided.

v65.0.0

No release notes provided.

v64.0.3

No release notes provided.

v64.0.2

No release notes provided.

v64.0.1

No release notes provided.

v64.0.0

No release notes provided.

v63.4.3

No release notes provided.

v63.4.2

No release notes provided.

... (truncated)

Changelog

Sourced from setuptools's changelog.

v65.5.1

Misc ^^^^

  • #3638: Drop a test dependency on the mock package, always use :external+python:py:mod:unittest.mock -- by :user:hroncok
  • #3659: Fixed REDoS vector in package_index.

v65.5.0

Changes ^^^^^^^

  • #3624: Fixed editable install for multi-module/no-package src-layout projects.
  • #3626: Minor refactorings to support distutils using stdlib logging module.

Documentation changes ^^^^^^^^^^^^^^^^^^^^^

  • #3419: Updated the example version numbers to be compliant with PEP-440 on the "Specifying Your Project’s Version" page of the user guide.

Misc ^^^^

  • #3569: Improved information about conflicting entries in the current working directory and editable install (in documentation and as an informational warning).
  • #3576: Updated version of validate_pyproject.

v65.4.1

Misc ^^^^

v65.4.0

Changes ^^^^^^^

v65.3.0

... (truncated)

Commits

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)
  • @dependabot use these labels will set the current labels as the default for future PRs for this repo and language
  • @dependabot use these reviewers will set the current reviewers as the default for future PRs for this repo and language
  • @dependabot use these assignees will set the current assignees as the default for future PRs for this repo and language
  • @dependabot use this milestone will set the current milestone as the default for future PRs for this repo and language

You can disable automated security fix PRs for this repo from the Security Alerts page.

Bumps [setuptools](https://github.com/pypa/setuptools) from 46.0.0 to 65.5.1.
- [Release notes](https://github.com/pypa/setuptools/releases)
- [Changelog](https://github.com/pypa/setuptools/blob/main/CHANGES.rst)
- [Commits](pypa/setuptools@v46.0.0...v65.5.1)

---
updated-dependencies:
- dependency-name: setuptools
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <[email protected]>
@dependabot dependabot bot added the dependencies Pull requests that update a dependency file label Dec 27, 2022
@mwetter
Copy link
Member

mwetter commented Jan 8, 2023

@JayHuLBL : can you please process this PR to apply this security patch. See the failed test, please make sure they pass (or drop python 2.7 if no longer needed).

Once fixed in funnel, please update the funnel version in BuildingsPy.

@JayHuLBL
Copy link
Contributor

JayHuLBL commented Jan 9, 2023

With gcc version 9.x, the command cmake --build . --target install run without issue. However, with gcc version 10.x and plus, a extern should be added for defining global variable. See lastpass/lastpass-cli#532. Otherwise, it will give error:

/usr/bin/ld: CMakeFiles/lib_obj.dir/mkdir_p.c.o:(.bss+0x0): multiple definition of `log_file'; 
CMakeFiles/lib_obj.dir/compare.c.o:(.bss+0x0): first defined here

Copy link
Member

@mwetter mwetter left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See inline comments.

setup.py Outdated
@@ -31,15 +31,14 @@
long_description=README,
long_description_content_type='text/markdown',
license="3-clause BSD",
python_requires='>=2.7, !=3.0.*, !=3.1.*, !=3.2.*, !=3.3.*, !=3.4.*, !=3.5.*',
python_requires='>2.7, !=3.0.*, !=3.1.*, !=3.2.*, !=3.3.*, !=3.4.*, !=3.5.*',
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In CMake (https://github.com/lbl-srg/funnel/pull/63/files#diff-cdd48abbd3eb8d1c54077449fc74a8de1f29805d2be5d8e5232b7aab76ea7a6fL16) and in requirements.txt Python 2.7 is no longer supported. Why is >2.7 here if the 2.x series is not tested?

src/readCSV.c Outdated
fgets(buf,100,fp); // skip the first "skipLines" lines
// skip the first "skipLines" lines
if (fgets(buf,100,fp) == NULL){
exit(1);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How is the error that leads to a non-zero exit code reported to the user? It looks like line 82 is true if the EOF is encountered or an error is encountered. This must be reported to stderr, no? If not, how would a user understand why the program exits with an error?

@mwetter
Copy link
Member

mwetter commented Jan 14, 2023

Regarding #63 (comment), if we drop Python 2.7, then this is not backward compatible and hence we need to update the version to 4.0.0. See https://semver.org/

@JayHuLBL
Copy link
Contributor

@mwetter I think this PR is ready to merge. Then should we release the tool as version 1.0.0, as the change drops Python 2.x? And then add the release new version number in BuildingsPy.

@JayHuLBL
Copy link
Contributor

The PR can be merged as the setuptools has been updated to 65.5.1. Also, the comments have been addressed through PR #66, in which the minimum Python version has been changed to 3.8.

@JayHuLBL JayHuLBL merged commit 0bb4b19 into master Sep 13, 2023
6 checks passed
@dependabot dependabot bot deleted the dependabot/pip/setuptools-65.5.1 branch September 13, 2023 22:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants