Skip to content

PYTHON-1411 Reproducibility fix for mongodb-python-driver - #343

Closed
shell-over-rice wants to merge 3 commits into
mongodb:masterfrom
shell-over-rice:master
Closed

shell-over-rice wants to merge 3 commits into
mongodb:masterfrom
shell-over-rice:master

Conversation

@shell-over-rice

Copy link
Copy Markdown
Contributor

The copyright message constructed depends on the system current date. So the build changes everytime(Every year technically) it runs. This dependency should not be there as part of reproducible-builds effort and has been fixed by removing the date alone.

Comment thread doc/conf.py Outdated
# General information about the project.
project = u'PyMongo'
copyright = u'MongoDB, Inc. 2008-{0}. MongoDB, Mongo, and the leaf logo are registered trademarks of MongoDB, Inc'.format(datetime.date.today().year)
copyright = u'MongoDB, Inc. MongoDB, Mongo, and the leaf logo are registered trademarks of MongoDB, Inc'

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I see. The date is still necessary, but it can be static. Change it to "2008-present" instead of removing it.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

ok will do that

Comment thread doc/conf.py
@@ -30,7 +30,7 @@

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Can you remove the now unused "import datetime" above?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

yaa sure