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

Fix issue 20021 - static if doesn't evaluate opCast(bool) #10237

Closed
wants to merge 258 commits into from

Conversation

dkorpel
Copy link
Contributor

@dkorpel dkorpel commented Jul 28, 2019

No description provided.

dlang-bot and others added 30 commits June 16, 2019 16:19
cppmangle: Follow up on dlang#10021
merged-on-behalf-of: Nicholas Wilson <thewilsonator@users.noreply.github.com>
Merge remote-tracking branch 'upstream/stable' into merge_stable
fix issue 19969 - Unhelpful error when attempting (incorrectly) to append to a string
merged-on-behalf-of: Nicholas Wilson <thewilsonator@users.noreply.github.com>
add -fPIE support to front end
Don't crash when compiling for 16-bit platforms
merged-on-behalf-of: Nicholas Wilson <thewilsonator@users.noreply.github.com>
Fix a regression spotted by SSoulaimane.
Also add the scoped Namespace tests as non-scoped tests.
cppmangle: Add more tests, fix double 'N'
merged-on-behalf-of: Nicholas Wilson <thewilsonator@users.noreply.github.com>
Version out assert in cppmangle
merged-on-behalf-of: Nicholas Wilson <thewilsonator@users.noreply.github.com>
elfobj.d: refactor with const and reltype_t
merged-on-behalf-of: Mathias LANG <pro.mathias.lang@gmail.com>
fix Issue 19646 - Initialization of globals not checked for @safe
…ymous unions (dlang#9909)

Fix Issue 19122 - Postblits and destructors called on members of anonymous unions
merged-on-behalf-of: unknown
elpicpie.d: carve out from elem.d
merged-on-behalf-of: Petar Kirov <ZombineDev@users.noreply.github.com>
dmd/utils.d: Remove extern(C++) linkage from internal helpers
merged-on-behalf-of: Walter Bright <WalterBright@users.noreply.github.com>
dmd/statement.h: Fix virtual hidden by overload errors
merged-on-behalf-of: Walter Bright <WalterBright@users.noreply.github.com>
dmd/root: Fix C++ headers to match new signatures
merged-on-behalf-of: Nicholas Wilson <thewilsonator@users.noreply.github.com>
dmd/module.h: Fix undefined reference to Module::read
merged-on-behalf-of: Walter Bright <WalterBright@users.noreply.github.com>
@dkorpel dkorpel changed the base branch from master to stable July 29, 2019 12:45
@dkorpel dkorpel changed the base branch from stable to master July 29, 2019 13:01
@dkorpel
Copy link
Contributor Author

dkorpel commented Jul 29, 2019

I tried targeting stable and rebasing with

git rebase --onto 062d2a origin/stable
git rebase --onto origin/stable master

But I'm still not familiar with rebasing, I don't think it worked out and now dlang-bot wants to auto-close 3 unrelated issues found in stable... How do you unlink issues?

@wilzbach wilzbach changed the base branch from master to stable July 29, 2019 13:11
@wilzbach
Copy link
Member

I tried targeting stable and rebasing with

git rebase --onto 062d2a origin/stable
git rebase --onto origin/stable master

But I'm still not familiar with rebasing, I don't think it worked out and now dlang-bot wants to auto-close 3 unrelated issues found in stable... How do you unlink issues?

You don't. You need to do a rebase onto upstream/stable. That's all.

dlang-bot and others added 10 commits July 30, 2019 01:53
redo fix for issue 11856: segfault with circular template constraints
merged-on-behalf-of: Nicholas Wilson <thewilsonator@users.noreply.github.com>
…namespace

The namespace was appended after the template declaration.
Fix 20094: Wrong C++ template substitution for arguments in the same namespace
merged-on-behalf-of: Nicholas Wilson <thewilsonator@users.noreply.github.com>
[stable]  Fixup PR 10256: Properly nest substituted C++ names in N..E
merged-on-behalf-of: Nicholas Wilson <thewilsonator@users.noreply.github.com>
@dkorpel
Copy link
Contributor Author

dkorpel commented Aug 6, 2019

Ok I don't get git rebasing at all, this PR has become such a mess that I'm going to close it and revive it. Sorry for the inconvenience.

@dkorpel dkorpel closed this Aug 6, 2019
@PetarKirov
Copy link
Member

PetarKirov commented Aug 7, 2019

@dkorpel The right way to go is the following:

  • Ensure you have the following 2 git remotes configured:
    • upstream, pointing to git@github.com:dlang/dmd.git
    • origin, pointing to your own fork of dmd, presumably git@github.com:dkorpel/dmd.git, from which you are going to open a pull request
    # Check which remotes you have configured:
    git remote -v
    
    # Add the following if missing:
    git remote add upstream git@github.com:dlang/dmd.git
    
    # If `git remote -v` had listed upstream, you need to
    # modify it, instead of adding it again:
    git remote set-url upstream git@github.com:dlang/dmd.git
    
    # (Do the same for origin.)
    
    # (If you don't have SSH Keys configured you may need to replace
    # those URLs with the https ones.)
  • Before doing anything, ensure that your local master and stable branches are up to date with upstream:
git checkout master
git pull --ff-only upstream master

git checkout stable
git pull --ff-only upstream stable
  • If you originally made your branch based of $BRANCH_SOURCE, and you need to rebase it on top of $BRANCH_DESTINATION, do the following:
# For your case the following may be the right branches:
BRANCH_SOURCE='master'
BRANCH_DESTINATION='upstream/stable'
PR_BRANCH='static-if-bool'

git checkout $PR_BRANCH

# If you rebased your source branch in the previous step, 
# you may need to rebase your PR branch as well:
git rebase $BRANCH_SOURCE

# And finally to retarget the destination branch:
git rebase --onto $BRANCH_DESTINATION $BRANCH_SOURCE

# (Now you may get merge conflicts. That's ok. When you resolve them you should be able
# to continue by doing `git rebase --continue`

# When done, all that's left is to publish your local branch to
# GitHub, by overwriting what was there before:
git push --force origin $PR_BRANCH

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

Successfully merging this pull request may close these issues.