Skip to content

* Fixes compilation issues - #10

Open
mohibsheth wants to merge 1 commit into
aaronweihe:masterfrom
mohibsheth:master
Open

* Fixes compilation issues#10
mohibsheth wants to merge 1 commit into
aaronweihe:masterfrom
mohibsheth:master

Conversation

@mohibsheth

Copy link
Copy Markdown

Fixes compilation issues with recent changes in csproj structure for Xamarin.Android

@aaronweihe

Copy link
Copy Markdown
Owner

I just fresh installed Xamarin Studio and it compiles fine from develop branch. I didn't install any Xamarin stuff prior on this computer. I doubt compiler errors you had was the issue of csproj file structure. Maybe it's a misconfig somewhere else on your side? Which version of Mono, Xamarin Android and Xamarin Studio you have?

@mohibsheth

Copy link
Copy Markdown
Author

The library project used to compile fine but Sample project didnt compile. No matter what I tried, Sample project didn't recognize CalendarPickerView. Once I had done the changes in csproj files to make it like an Android Library, it compiled fine.

I am on latest Stable branch. Xamarin Studio 5.7 / Xamarin Android 4.20.0.28

@aaronweihe

Copy link
Copy Markdown
Owner

hmmm. That's odd. I was able to run the sample on the first try. Where did you change to make it an Android Library? I couldn't find anything related to that. The only thing might be related to that is removing AndroidManifest, but Android library can totally have the manifest.

@mohibsheth

Copy link
Copy Markdown
Author

When I create a new Android library, I dont see a AndroidManifest. Also when you go to Project Options, I dont see options like "Android Build", "Android Application", "Android Package Signing" which I can totally see for the MonoDroid.TimesSquare library project. So something is definitely amiss.

Anyways, if you can compile it fine, no worries, but it would become difficult for me send PRs.

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.

2 participants