Skip to content
This repository was archived by the owner on Mar 24, 2026. It is now read-only.

Vibed#1606

Merged
msmith-techempower merged 22 commits intoremove-install-scriptsfrom
vibed
Jun 2, 2015
Merged

Vibed#1606
msmith-techempower merged 22 commits intoremove-install-scriptsfrom
vibed

Conversation

@msmith-techempower
Copy link
Copy Markdown
Member

Introduces vibed based on the initial commit from @nin-jin in #1597

This simply fixed the minor bugs and removed the install.sh file.

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.

I think xdg-utils is not necessary.
@nin--jin said

Note that the line sudo apt-get install -qq -y g++ gcc-multilib xdg-utils shouldn't be necessary.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Good point; I'll try removing it from my system and re-running the test.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Checked - it is required.

We needed to update and specify the correct DFLAGS in
the dmd.conf file. Additionally, we had to copy this
file to the appropriate directory to be automatically
loaded by dmd.
@msmith-techempower
Copy link
Copy Markdown
Member Author

Well... travis failed. However, I have re-tested this on my local several times and it just works... not sure what the deal is... inclined to merge as-is.

msmith-techempower added a commit that referenced this pull request Jun 2, 2015
@msmith-techempower msmith-techempower merged commit 6a97d74 into remove-install-scripts Jun 2, 2015
@msmith-techempower msmith-techempower deleted the vibed branch June 2, 2015 20:30
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants