Thread Rating:
  • 0 Vote(s) - 0 Average
  • 1
  • 2
  • 3
  • 4
  • 5
CDC_interrupt IDE V12
31-07-2018, 11:25 AM, (This post was last modified: 31-07-2018, 11:25 AM by 2e0byo. Edit Reason: typo )
#1
CDC_interrupt IDE V12
I thought I'd leave this lying around here as it took me a while to figure out:

With the current (v12) ide and libraries installed by the auto-install script* there's a typo in p8/include/pinguino/core/usbcdc.c line 376 should read CDC_interrupt not cdc_interrupt, or compilation fails.

I'd submit a bug report or preferably just a pull request, but the github pages don't seem to match the installed files at all (or maybe I'm being stupid?)

Anyone know where the v12 sources are kept?

There's also a bug in the PWM library at least for the 18f4550 but I've not got round to looking into it yet.  analogueWrite is fine so it all should work.

Ta all,

2e0byo


*after editing pinguino-ide.deb not to depend on python-pyside which is available through pip but not (on current) debian testing from the package manager: a few people are posting 'file not found' after the installer script which is caused by dpkg skipping the actual IDE.
Reply
10-10-2018, 09:33 AM,
#2
RE: CDC_interrupt IDE V12
(31-07-2018, 11:25 AM)Hi 2e0byo, Wrote: With the current (v12) ide and libraries installed by the auto-install script* there's a typo in p8/include/pinguino/core/usbcdc.c line 376 should read CDC_interrupt not cdc_interrupt, or compilation fails.

> Actually the type is in main.c it should be cdc_interrupt() instead of CDC_interrupt()

I'd submit a bug report or preferably just a pull request, but the github pages don't seem to match the installed files at all (or maybe I'm being stupid?)

> Look at the v12-dev branch not in the master (v11) one.

There's also a bug in the PWM library at least for the 18f4550 but I've not got round to looking into it yet.  analogueWrite is fine so it all should work.

> Do you have an example that doesn't work ?

*after editing pinguino-ide.deb not to depend on python-pyside which is available through pip but not (on current) debian testing from the package manager: a few people are posting 'file not found' after the installer script which is caused by dpkg skipping the actual IDE.

> The deb package was expecting pyside to have a version >=1.1.0-0, the new deb package will have no restriction and I hope it will be enough.
It is easier to complain than it is to do, but it is better to do than it is to complain.
Reply
10-10-2018, 12:02 PM, (This post was last modified: 10-10-2018, 12:03 PM by 2e0byo.)
#3
RE: CDC_interrupt IDE V12
Pinguino isn't dead! I didn't expect a reply to that..

I'll have a look through github.

On debian testing, pyside is not longer available (although there are plenty of pyside2-xyz modules), but one can still install it from pip.

Re. PWM, I'm miles from a pinguino at the moment: will look into it when term is over.

--2e0byo
Reply


Forum Jump:


Users browsing this thread: 1 Guest(s)