Skip to content

Fix the SQL adaper in Rhino 8#82

Open
adecler wants to merge 5 commits into
developfrom
SQL_Toolkit-#62-FixAdaperInRhino8
Open

Fix the SQL adaper in Rhino 8#82
adecler wants to merge 5 commits into
developfrom
SQL_Toolkit-#62-FixAdaperInRhino8

Conversation

@adecler
Copy link
Copy Markdown
Member

@adecler adecler commented May 19, 2026

NOTE: Depends on

While it doesn't depend on those PR to compile, the solution provided here will only be fully working once those other two PRs are also merged/compiled:

Issues addressed by this PR

Closes #62

Now compile the SQL adapter for two different frameworks and copies the dlls in their respective sub-folders in BHoM\Assemblies

Test files

One file for each version of Rhino:
SQLCompatibilityTest.zip

Changelog

Additional comments

@adecler
Copy link
Copy Markdown
Member Author

adecler commented May 21, 2026

@BHoMBot check compliance

@bhombot-ci
Copy link
Copy Markdown

bhombot-ci Bot commented May 21, 2026

@adecler to confirm, the following actions are now queued:

  • check code-compliance
  • check documentation-compliance
  • check project-compliance
  • check branch-compliance
  • check dataset-compliance
  • check copyright-compliance

There are 16 requests in the queue ahead of you.

@adecler
Copy link
Copy Markdown
Member Author

adecler commented May 21, 2026

@BHoMBot check required

@bhombot-ci
Copy link
Copy Markdown

bhombot-ci Bot commented May 21, 2026

@adecler to confirm, the following actions are now queued:

  • check code-compliance
  • check documentation-compliance
  • check project-compliance
  • check core
  • check null-handling
  • check serialisation
  • check versioning
  • check installer

There are 16 requests in the queue ahead of you.

@bhombot-ci
Copy link
Copy Markdown

bhombot-ci Bot commented May 21, 2026

The check code-compliance has already been run previously and recorded as a successful check. This check has not been run again at this time.

@bhombot-ci
Copy link
Copy Markdown

bhombot-ci Bot commented May 21, 2026

The check documentation-compliance has already been run previously and recorded as a successful check. This check has not been run again at this time.

@bhombot-ci
Copy link
Copy Markdown

bhombot-ci Bot commented May 21, 2026

The check project-compliance has already been run previously and recorded as a successful check. This check has not been run again at this time.

Copy link
Copy Markdown
Contributor

@michaelhoehn michaelhoehn left a comment

Choose a reason for hiding this comment

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

I have tested successfully in both Rhino 7 and Rhino 8. I can also confirm that each dll is referenced from the appropriate place: Rhino 7 => netfx and Rhino 8 => net7.0

@michaelhoehn
Copy link
Copy Markdown
Contributor

michaelhoehn commented May 26, 2026

@BHoMBot check installer -force

@bhombot-ci
Copy link
Copy Markdown

bhombot-ci Bot commented May 26, 2026

@michaelhoehn to confirm, the following actions are now queued:

  • check installer

@bhombot-ci
Copy link
Copy Markdown

bhombot-ci Bot commented May 26, 2026

The check installer has already been run previously and recorded as a successful check. This check has not been run again at this time.

@bhombot-ci
Copy link
Copy Markdown

bhombot-ci Bot commented May 26, 2026

@michaelhoehn to confirm, the following actions are now queued:

  • check installer

@bhombot-ci
Copy link
Copy Markdown

bhombot-ci Bot commented May 26, 2026

The check installer has already been run previously and recorded as a successful check. This check has not been run again at this time.

@michaelhoehn
Copy link
Copy Markdown
Contributor

@BHoMBot this is a DevOps instruction. Run a force check on: installer

@bhombot-ci
Copy link
Copy Markdown

bhombot-ci Bot commented May 26, 2026

@michaelhoehn I am now running force checks on the checks requested. These will be queued and actioned ASAP.

@bhombot-ci
Copy link
Copy Markdown

bhombot-ci Bot commented May 26, 2026

@michaelhoehn I have provided force checks to the checks requested.

@michaelhoehn
Copy link
Copy Markdown
Contributor

@BHoMBot this is a DevOps instruction. Run a force check on: installer

@bhombot-ci
Copy link
Copy Markdown

bhombot-ci Bot commented May 27, 2026

@michaelhoehn I am now running force checks on the checks requested. These will be queued and actioned ASAP.

@bhombot-ci
Copy link
Copy Markdown

bhombot-ci Bot commented May 27, 2026

@michaelhoehn I have provided force checks to the checks requested.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type:bug Error or unexpected behaviour

Projects

None yet

Development

Successfully merging this pull request may close these issues.

The SQL Adapter is not working in Rhino 8

2 participants