Skip to content

Repair HASH_CODE support and enable it in Travis #2044

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

Closed
wants to merge 2 commits into from

Conversation

tautschnig
Copy link
Collaborator

See individual commit messages.

@smowton
Copy link
Contributor

smowton commented Apr 12, 2018

Why are you adding the new flag specifically to OSX/Clang?

@tautschnig
Copy link
Collaborator Author

Why are you adding the new flag specifically to OSX/Clang?

I just wanted to re-use some existing one rather than introducing a new stage. Would one of the others be better suited?

@smowton
Copy link
Contributor

smowton commented Apr 12, 2018

Ah, I see "one travis config," so I guess that's intentional. Still, it muddies the waters rather -- when it fails, is that an OSX problem, a Clang problem, or a HASH_CODE problem?

I suggest to deal with the myriad of non-default options, instead of piling them on different configurations in a somewhat arbitrary fashion, we should adopt a weekly (daily?) Travis cron job and use conditional builds (https://docs.travis-ci.com/user/conditional-builds-stages-jobs/) to enable these unusual configurations only in the cron environment.

This will mean PRs can appear fine and get merged if they break an odd configuration (boo!), but they consume acceptable resources while we eventually get notified of the obscure breakage, unlike some peculiar configurations which currently often break without us noticing at all -- e.g. !USE_DSTRING, or SUB_IS_LIST.

@smowton
Copy link
Contributor

smowton commented Apr 12, 2018

@peterschrammel thoughts?

@tautschnig
Copy link
Collaborator Author

I'm all for doing those via cron jobs, but where and how is one notified when something breaks?

@smowton
Copy link
Contributor

smowton commented Apr 12, 2018

Per default Travis will always notify the committer and commit author of the tip of the broken branch, regardless of the build type, though obviously whoever happens to be the most recent committer is pretty arbitrary. We also have a webhook notification set up (a la https://docs.travis-ci.com/user/notifications/#Webhooks-Delivery-Format) which would give good visibility of a breakage in the office.

We could easily add a gateway to that webhook to also mail <list of nominated people> when the failing build has "type: cron".

The existing regression tests would have uncovered the problem if only
they had been used in this configuration.
@tautschnig
Copy link
Collaborator Author

I have moved the Travis check to cron in this PR and have similarly cleaned up other Travis configurations in #2047.

@tautschnig
Copy link
Collaborator Author

Closing as HASH_CODE is now enabled by default (with #1992 merged).

@tautschnig tautschnig closed this May 12, 2019
@tautschnig tautschnig deleted the irep-hash-code-fix branch May 12, 2019 17:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants