Skip to content

fix issue 17198 - rdmd does not recompile when --extra-file is added#218

Closed
aG0aep6G wants to merge 1 commit intodlang:masterfrom
aG0aep6G:17198
Closed

fix issue 17198 - rdmd does not recompile when --extra-file is added#218
aG0aep6G wants to merge 1 commit intodlang:masterfrom
aG0aep6G:17198

Conversation

@aG0aep6G
Copy link
Contributor

No description provided.

@dlang-bot
Copy link
Contributor

Fix Bugzilla Description
17198 rdmd does not recompile when --extra-file is added

@dlang-bot dlang-bot added the Type.Bug Things don't work as they were intended to, or the way they were intended to work doesn't make sense label Feb 20, 2017
@wilzbach
Copy link
Contributor

@aG0aep6G to iterate what Martin is trying to say for the last year:

All bug fixes should go into stable

(for the point releases the stable branch will be used)

We are currently trying to work out a way to let the DLang-Bot warn you in such cases - see e. g. dlang/dlang-bot#8 if you are interested.

Another idea would be to let the Dlang-Bot trigger a similar PR for stable once a bug fix arrives in master (or the other way around): https://github.com/dlang-bots/dlang-bot/issues

@aG0aep6G
Copy link
Contributor Author

All bug fixes should go into stable

Ugh, GitHub should really let me change the target branch of a pull request. It's so easy to forget. Anyway, here it is against stable: #219.

@aG0aep6G aG0aep6G closed this Feb 20, 2017
@wilzbach
Copy link
Contributor

@aG0aep6G
Copy link
Contributor Author

They do since last summer: https://github.com/blog/2224-change-the-base-branch-of-a-pull-request

Double dukes. You have to click the "Edit" button next to the title to make that drop-down list appear. Didn't occur to me. Well, let's just go with #219 now :|

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Type.Bug Things don't work as they were intended to, or the way they were intended to work doesn't make sense

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants