Skip to content

Updated Project Build Scripts and Visual Studio Build#9

Merged
EPTamminga merged 2 commits into
DNNCommunity:masterfrom
SkyeHoefling:build
Jul 9, 2018
Merged

Updated Project Build Scripts and Visual Studio Build#9
EPTamminga merged 2 commits into
DNNCommunity:masterfrom
SkyeHoefling:build

Conversation

@SkyeHoefling

@SkyeHoefling SkyeHoefling commented Jul 8, 2018

Copy link
Copy Markdown

Updated Build Scripts to decouple the project from a Dnn Installation. Building within a Dnn Installation is an older way of module development. Most developer prefer to keep their module development decoupled. This change also makes it easier for developers to make more changes to this module since they can just download the source code and build.

Changes made

  • Removed all references DotNetNuke.* references and ../../bin of local dnn installation. The DotNetNuke.* references have been updated to use NuGet version 8.0.4. I picked this build as the oldest v8 build of Dnn we can change this to whatever is the minimum supported version of this module
  • Updated NET Framework to version 4.5, we were having build issues with the DotNetNuke.* dlls since it requires 4.5
  • Removed hardcoded references to the MSBuild Tasks and added NuGet Reference
  • Updated bin directories to be bin/Debug and bin/Release instead of ../../bin
  • Updated gitignore

PR Template Checklist

  • Other - Fixes local build and install issues

Testing

After making these changes and getting it to build I was able to take the install file generated and install the module into a Dnn Sandbox Instance I have running Dnn v9.2

Follow Up

Since we have isolated the build from a Dnn Installation. If this Pull Request is approved we could easily set up an AppVeyor build to certify Pull Requests.

Andrew Hoefling added 2 commits July 7, 2018 22:53
…asier; Upgraded netframework to .net 4.5 since Dnn is using 4.5; Updated all DotNetNuke references to use NuGet instead of local dnn instance; Updated bin director to be local instead of using dnn instance - bin/release and bin/debug; Added MSBuild Tasks NuGet so it is not dependent on the Tests tasks
@valadas

valadas commented Jul 8, 2018

Copy link
Copy Markdown
Member

I love it, I have been trying to do the same on all modules I have touched. I will test and merge soon. I have no experience with appveyor, does the organization have access to that free, or is it tied to a user account, how does that work? Would be nice to implement on the other modules too.

@SkyeHoefling

Copy link
Copy Markdown
Author

We should be able to set something up pretty easy that is linked right to this repo. AppVeyor offers free builds for anyone building open source software. For the small modules such as this I think it is a great way to get a simple build running on Pull Requests.

Some Options:

  • A project maintainer could create the build with their github/appveyor account
  • Create a AppVeyor account with the DnnCommunity that is shared.

I really don't think it matters either way and I would just go with option 1 for now and iterated on it over time.

Let me know if there is anything else you need me to do on this

@EPTamminga EPTamminga left a comment

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 this is good to go.

@EPTamminga
EPTamminga merged commit 3e7ae44 into DNNCommunity:master Jul 9, 2018
@SkyeHoefling

Copy link
Copy Markdown
Author

@EPTamminga @valadas Did anyone try building this locally? I think there may be an issue with the BuildScripts folder that was hidden by the Visual Studio Cache. I started running into some issues this morning when I tried making some changes

@valadas

valadas commented Jul 11, 2018

Copy link
Copy Markdown
Member

@ahoefling Yeah, it is looking for the msbuild dll in the wrong folder, let me fix that...

@valadas

valadas commented Jul 11, 2018

Copy link
Copy Markdown
Member

Fixed and merged with #13

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.

3 participants