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

Change compilation settings to only do x86 and hide unnecessary COM object #1

Merged
merged 2 commits into from
Nov 6, 2012

Conversation

danielcaldwell
Copy link
Contributor

Changed project settings to compile for x86 only, also removed the com
visibility of OGRAddLayerDialog as it's not a com object.

Changed project settings to compile for x86 only, also removed the com
visibility of OGRAddLayerDialog as it's not a com object.
rburhum added a commit that referenced this pull request Nov 6, 2012
Change compilation settings to only do x86 and hide unnecessary COM object
@rburhum rburhum merged commit 9db0398 into rburhum:master Nov 6, 2012
@rburhum
Copy link
Owner

rburhum commented Nov 6, 2012

yeah. you can see it under Files Changed. Thanks man!

@danielcaldwell
Copy link
Contributor Author

Sounds good, I must have earned at least 1 karma point for this... lemme
know if you want my help with anything else...

On Mon, Nov 5, 2012 at 4:53 PM, Ragi Yaser Burhum
[email protected]:

yeah. you can see it under Files Changed. Thanks man!


Reply to this email directly or view it on GitHubhttps://github.com//pull/1#issuecomment-10095073.

@rburhum
Copy link
Owner

rburhum commented Nov 6, 2012

Yup man! Well, according to the latest SP1 release notes (NIM083294 from http://downloads.esri.com/support/downloads/other_/ArcGIS101sp1-issues.htm) There is write support now. http://resources.arcgis.com/en/help/arcobjects-net/componenthelp/index.html#/IsReadOnly_Property/00250000068v000000/ but the docs have not been updated. If you ever feel bored, you could implement write support hahaha. Or not. Thanks anyway man

@danielcaldwell
Copy link
Contributor Author

So with Open Source Projects like these, what do you do to support 10.1 and
10.1 sp1? Do you only support SP1 and tell everyone to upgrade? Or do you
fork or something?

On Mon, Nov 5, 2012 at 6:18 PM, Ragi Yaser Burhum
[email protected]:

Yup man! Well, according to the latest SP1 release notes (NIM083294 from
http://downloads.esri.com/support/downloads/other_/ArcGIS101sp1-issues.htm)
There is write support now.
http://resources.arcgis.com/en/help/arcobjects-net/componenthelp/index.html#/IsReadOnly_Property/00250000068v000000/but the docs have not been updated. If you ever feel bored, you could
implement write support hahaha. Or not. Thanks anyway man


Reply to this email directly or view it on GitHubhttps://github.com//pull/1#issuecomment-10096963.

@rburhum
Copy link
Owner

rburhum commented Nov 6, 2012

It's your choice. I would basically create a branch and leave the read-only support for 10.x+ and trunk would move to SP1 with write support. If really needed, we always have the ability to port patches to the previous branch. And with git, it is relatively easy

@danielcaldwell
Copy link
Contributor Author

I don't see any additional methods. Is there a interface I'm missing? Maybe
they didn't reship the interop or something? Not sure.

On Tue, Nov 6, 2012 at 11:26 AM, Ragi Yaser Burhum <[email protected]

wrote:

It's your choice. I would basically create a branch and leave the
read-only support for 10.x+ and trunk would move to SP1 with write support.
If really needed, we always have the ability to port patches to the
previous branch. And with git, it is relatively easy


Reply to this email directly or view it on GitHubhttps://github.com//pull/1#issuecomment-10124017.

@rburhum
Copy link
Owner

rburhum commented Nov 8, 2012

I had an e-mail exchange with Brent. He told me to implement ITable and ITableWrite

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