Skip to content

Conversation

andralex
Copy link
Member

@andralex andralex commented Mar 7, 2013

This was a design issue, which this diff fixes by adding a new file to track the last build time.

@andralex
Copy link
Member Author

andralex commented Mar 7, 2013

@andralex
Copy link
Member Author

andralex commented Mar 7, 2013

wondering why github didn't automatically notify bugzilla...

@ghost
Copy link

ghost commented Mar 7, 2013

It's notified when the pull is merged.

@ghost
Copy link

ghost commented Mar 7, 2013

LGTM, although it's a shame we don't have a test-suite for RDMD. I guess we could work on this some time later.

@andralex
Copy link
Member Author

andralex commented Mar 7, 2013

Cool. You should know you get to merge, too :o).

@ghost
Copy link

ghost commented Mar 7, 2013

Cool. You should know you get to merge, too :o).

I'm waiting for it to become official, you did say you were going to publish results about the community vote.

@andralex
Copy link
Member Author

andralex commented Mar 7, 2013

Destroy. http://goo.gl/LPFSL

@andralex
Copy link
Member Author

andralex commented Mar 7, 2013

I agree a test suite for rdmd would be awesome.

@ghost
Copy link

ghost commented Mar 7, 2013

I agree a test suite for rdmd would be awesome.

I'll work on it then.

@andralex
Copy link
Member Author

andralex commented Mar 7, 2013

@ghost
Copy link

ghost commented Mar 7, 2013

Thanks, I'll add this as a test-case later.

ghost pushed a commit that referenced this pull request Mar 7, 2013
Fix Issue 4814 - rdmd: Doesn't rebuild when using -of and turning an -L linker option on or off
@ghost ghost merged commit e14e037 into dlang:master Mar 7, 2013
This pull request was closed.
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.

1 participant