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

Build system doesn't detect modified files #264

Open
MihaMarkic opened this issue Jan 13, 2017 · 8 comments
Open

Build system doesn't detect modified files #264

MihaMarkic opened this issue Jan 13, 2017 · 8 comments

Comments

@MihaMarkic
Copy link

"Build Solution" won't detect modified files and won't compile anything if there is an earlier compilation available. "Rebuild Solution" will work.

Steps to reproduce:

Create a new project, compile it, change output from Hello World! to something else, do a Build Solution and run it.

Expected behavior:

New text should be visible.

Actual behavior:

"Hello world!" is displayed.

System

VS2015, rustc 1.14.0, VR 0.1.2

@Vort
Copy link

Vort commented Feb 16, 2017

Project build - the main feature of the plugin - is broken and no one cares about it? That's strange.

@Vbif
Copy link
Contributor

Vbif commented Feb 16, 2017

Try last release https://ci.appveyor.com/project/vosen/visualrust/build/145/artifacts . It use cargo to build a project

@Vort
Copy link

Vort commented Feb 16, 2017

@Vbif, MSBuild.exe crashes with System.NullReferenceException while trying to build default "Hello, world!" project.
I can provide additional information if needed.

@MaulingMonkey
Copy link
Collaborator

I fixed one MSBuild NullReferenceException crash which I suspect was @Vort 's issue in #286 . Unofficial test build @ https://ci.appveyor.com/project/MaulingMonkey/visualrust/build/1/artifacts , no idea if it's appropriately codesigned or not.

Changes to the project are showing up in builds (e.g. @MihaMarkic 's original bug is fixed?) for me as of:
VS2015 u3
rustc 1.18.0 (03fc9d622 2017-06-06)
cargo 0.19.0 (28d1d60d4 2017-05-16)
Visual Rust "0.2" ( https://github.com/MaulingMonkey/VisualRust/tree/e070ed2e1104a049d5a6119761d760b1b7d8a879 )

@Vort
Copy link

Vort commented Jul 26, 2017

@MaulingMonkey, I've tested that build with VS 2015 and Rust 1.19.0.
Looks like problem is gone.
But this changes should be released to become available for users, right?

@MaulingMonkey
Copy link
Collaborator

I hope so, but so far nobody's reviewed my pull request.

@Boddlnagg
Copy link
Contributor

The maintainer of this project is @vosen, and he's also the only one who can release new versions. Unfortunately I do not have time right now to review myself, and also I don't know enough about vosen's plans for the future direction of some features of VisualRust, making it hard for me to review any non-trivial PRs.

But anyway, thank you very much for your PR, @MaulingMonkey! 👍

@MaulingMonkey
Copy link
Collaborator

I did notice @vosen shows some activity every month or two, so I'm hopeful my changes might at least get reviewed in the next cycle. And I'm assuming this is largely volunteer work or side projects, so it's completely understandable to be pressed for time, and to have higher priorities elsewhere 😉. Worst case scenario I can start more actively maintaining my own fork as a longer term thing - I just want to avoid it becoming another permanent fork if it can at all be helped.

Thanks for the kind words, and your work which I'm building on, @Boddlnagg 👍

@MaulingMonkey MaulingMonkey added this to the 0.2 milestone Sep 20, 2017
@MaulingMonkey MaulingMonkey self-assigned this Sep 20, 2017
@MaulingMonkey MaulingMonkey removed their assignment Jul 25, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

No branches or pull requests

5 participants