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

ValueError: Expecting Property Name on compile #759

Closed
cosmikwolf opened this issue Aug 22, 2016 · 17 comments
Closed

ValueError: Expecting Property Name on compile #759

cosmikwolf opened this issue Aug 22, 2016 · 17 comments
Assignees
Labels
Milestone

Comments

@cosmikwolf
Copy link

I can successfully install pio 3.0.0a6, but when I try to build, i get the following errors:

MacPro:flxs1 tenkai$ pio run -v
[Mon Aug 22 13:37:51 2016] Processing teensy31 (platform: teensy, board_f_cpu: 96000000, build_flags: , board: teensy31, framework: arduino)

ValueError: Expecting property name: line 11 column 1 (char 291):
File "/Users/tenkai/.atom/packages/platformio-ide/penv/lib/python2.7/site-packages/platformio/builder/main.py", line 125:

env.SConscript("$BUILD_SCRIPT")
File "/Users/tenkai/.platformio/packages/tool-scons/script/../engine/SCons/Script/SConscript.py", line 551:
return SConscript(self.fs, *files, **substkw)
File "/Users/tenkai/.platformio/packages/tool-scons/script/../engine/SCons/Script/SConscript.py", line 260:
exec file in call_stack[-1].globals
File "/Users/tenkai/.platformio/platforms/teensy/builder/main.py", line 199:
target_elf = env.BuildProgram()
File "/Users/tenkai/.platformio/packages/tool-scons/script/../engine/SCons/Environment.py", line 224:
return self.method(*nargs, **kwargs)
File "/Users/tenkai/.atom/packages/platformio-ide/penv/lib/python2.7/site-packages/platformio/builder/tools/platformio.py", line 61:
deplibs = env.BuildDependentLibraries("$PROJECTSRC_DIR")
File "/Users/tenkai/.platformio/packages/tool-scons/script/../engine/SCons/Environment.py", line 224:
return self.method(*nargs, **kwargs)
File "/Users/tenkai/.atom/packages/platformio-ide/penv/lib/python2.7/site-packages/platformio/builder/tools/piolib.py", line 514:
lib_builders = env.GetLibBuilders()
File "/Users/tenkai/.platformio/packages/tool-scons/script/../engine/SCons/Environment.py", line 224:
return self.method(*nargs, **kwargs)
File "/Users/tenkai/.atom/packages/platformio-ide/penv/lib/python2.7/site-packages/platformio/builder/tools/piolib.py", line 489:
lb = LibBuilderFactory.new(env, join(libs_dir, item))
File "/Users/tenkai/.atom/packages/platformio-ide/penv/lib/python2.7/site-packages/platformio/builder/tools/piolib.py", line 50:
obj = getattr(sys.modules[name], clsname)(env, path)
File "/Users/tenkai/.atom/packages/platformio-ide/penv/lib/python2.7/site-packages/platformio/builder/tools/piolib.py", line 85:
self.manifest = manifest if manifest else self.loadmanifest()
File "/Users/tenkai/.atom/packages/platformio-ide/penv/lib/python2.7/site-packages/platformio/builder/tools/piolib.py", line 393:
manifest = util.load_json(join(self.path, "library.json"))
File "/Users/tenkai/.atom/packages/platformio-ide/penv/lib/python2.7/site-packages/platformio/util.py", line 132:
return json.load(f)
File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/json/__init__.py", line 290:
**kw)
File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/json/__init__.py", line 338:
return defaultdecoder.decode(s)
File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/json/decoder.py", line 366:
obj, end = self.raw_decode(s, idx=_w(s, 0).end())
File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/json/decoder.py", line 382:
obj, end = self.scan_once(s, idx)
============================================================================================== [ERROR] Took 0.49 seconds ==============================================================================================

This is compiling for Teensy version 1.28 (latest teensy version on pio)

@ivankravets ivankravets added this to the 3.0.0 milestone Aug 22, 2016
@ivankravets ivankravets self-assigned this Aug 22, 2016
@ivankravets
Copy link
Member

Please reinstall (repeat pip install) PlatformIO 3.0 http://docs.platformio.org/en/stable/installation.html#development-version

You should see warning/yellow message about a broken library. This library contains library.json. Could you share it here? Thanks.

@cosmikwolf
Copy link
Author

I uninstalled and reinstalled the dev version and I did not get any errors about any libraries.

  Successfully uninstalled platformio-2.11.2
MacPro:flxs1 tenkai$ pip install -U https://github.com/platformio/platformio/archive/develop.zip
Collecting https://github.com/platformio/platformio/archive/develop.zip
  Downloading https://github.com/platformio/platformio/archive/develop.zip
     - 11.8MB 4.9MB/s
Requirement already up-to-date: bottle<0.13 in /Users/tenkai/.atom/packages/platformio-ide/penv/lib/python2.7/site-packages (from platformio==3.0.0a6)
Requirement already up-to-date: click<6,>=5 in /Users/tenkai/.atom/packages/platformio-ide/penv/lib/python2.7/site-packages (from platformio==3.0.0a6)
Requirement already up-to-date: lockfile<0.13,>=0.9.1 in /Users/tenkai/.atom/packages/platformio-ide/penv/lib/python2.7/site-packages (from platformio==3.0.0a6)
Requirement already up-to-date: requests<3,>=2.4.0 in /Users/tenkai/.atom/packages/platformio-ide/penv/lib/python2.7/site-packages (from platformio==3.0.0a6)
Requirement already up-to-date: semantic_version>=2.5.0 in /Users/tenkai/.atom/packages/platformio-ide/penv/lib/python2.7/site-packages (from platformio==3.0.0a6)
Requirement already up-to-date: colorama in /Users/tenkai/.atom/packages/platformio-ide/penv/lib/python2.7/site-packages (from platformio==3.0.0a6)
Requirement already up-to-date: pyserial<4 in /Users/tenkai/.atom/packages/platformio-ide/penv/lib/python2.7/site-packages (from platformio==3.0.0a6)
Installing collected packages: platformio
  Running setup.py install for platformio ... done
Successfully installed platformio-3.0.0a6

@ivankravets
Copy link
Member

I mean "embedded" library, not system. Please process/run your project.

@cosmikwolf
Copy link
Author

I ran the project, and i get a bunch of errors, but the first one is this one:
Skip library with broken manifest: /Users/tenkai/.platformio/packages/framework-arduinoteensy/libraries/Artnet

here is the library.json file:

MacPro:Artnet tenkai$ pwd
/Users/tenkai/.platformio/packages/framework-arduinoteensy/libraries/Artnet
MacPro:Artnet tenkai$ cat library.json 
{
"name": "ArtNet",
"frameworks": "Arduino",
"keywords": "ArtNet, LED, LEDs, DMX, E1.31", 
"description": "Control LEDs over Ethernet using the Artnet Protocol",
"url": "http://forum.pjrc.com/threads/24688-Artnet-to-OctoWS2811/page10",
"authors":
[
{
        "name": "Nathanaël Lécaudé",
},
{
        "name": "mortonkopf",
}
]
"repository":
{
    "name": "git",
    "url": "https://github.com/natcl/Artnet"
}
}

@ivankravets
Copy link
Member

I got, the problem is linked with unicode

@ivankravets ivankravets reopened this Aug 22, 2016
@ivankravets
Copy link
Member

ivankravets commented Aug 22, 2016

Where did you find this manifest? It's broken.

I've just fixed it:

{  
  "name":"ArtNet",
  "frameworks":"Arduino",
  "platforms": "teensy",
  "keywords":"ArtNet, LED, LEDs, DMX, E1.31",
  "description":"Control LEDs over Ethernet using the Artnet Protocol",
  "homepage":"http://forum.pjrc.com/threads/24688-Artnet-to-OctoWS2811/page10",
  "authors":[  
    {  
      "name":"Nathanaël Lécaudé"
    },
    {  
      "name":"mortonkopf"
    }
  ],
  "repository":{  
    "name":"git",
    "url":"https://github.com/natcl/Artnet"
  }
}

Please use JSON validator https://jsonformatter.curiousconcept.com

@cosmikwolf
Copy link
Author

It is what was installed with the framework-teensy that gets installed with platformio.

@cosmikwolf
Copy link
Author

ah! no, I figued that framework-arduinoteensy was re-installed with platformio. This is the framework-arduinoteensy that comes from the PJRC installer. I had updated it to 1.29 to try and troubleshoot another issue. So the broken unicode must come from PJRC.

@cosmikwolf
Copy link
Author

I deleted framework-arduinoteensy and re-ran my project and now it seems to work. Lots of other new errors, but I need to work through them.
Thanks!

@ivankravets
Copy link
Member

It's not broken unicode. The unicode is OK. The problem with JSON format. It's broken.

I've just checked framework-arduinoteensy and don't see this library:

tree -L 1 /Users/ikravets/.platformio/packages/framework-arduinoteensy/libraries
/Users/ikravets/.platformio/packages/framework-arduinoteensy/libraries
├── Audio
├── Bounce
├── Bridge
├── EEPROM
├── Esplora
├── Ethernet
├── Firmata
├── LiquidCrystal
├── RobotIRremote
├── Robot_Control
├── Robot_Motor
├── SD
├── SPI
├── Scheduler
├── Servo
├── SoftwareSerial
├── Stepper
├── TFT
├── Temboo
├── USBHost
├── WiFi
└── Wire

Is this new library located in 1.29?

@ivankravets
Copy link
Member

Lots of other new errors, but I need to work through them.

This is PlatformIO related issues or yours?

@cosmikwolf
Copy link
Author

I think it might be platform related, but I am not sure:

In file included from /Users/tenkai/.platformio/packages/framework-arduinoteensy/libraries/Audio/src/Audio.h:16:0,
from /Users/tenkai/Desktop/Projects/flxs1/src/flxs1.ino:1:
/Users/tenkai/.platformio/packages/framework-arduinoteensy/libraries/Audio/src/DAC.h:21:16: error: expected ')' before '*' token
DACClass(Dacc *_dac, uint32_t _dacId, IRQn_Type _isrId) :

@cosmikwolf
Copy link
Author

Lots of other errors, but that seems to be the root

@cosmikwolf
Copy link
Author

And yes, that new library is in 1.29

@ivankravets
Copy link
Member

😆

name=Audio
version=1.0
author=Arduino
maintainer=Arduino <info@arduino.cc>
sentence=Allows playing audio files from an SD card. For Arduino DUE only.
paragraph=With this library you can use the Arduino Due DAC outputs to play audio files.<br />The audio files must be in the raw .wav format.
category=Signal Input/Output
url=http://arduino.cc/en/Reference/Audio
architectures=sam

What does this library do in Teensy framework? Could you compile it with Teeensyduino?

@cosmikwolf
Copy link
Author

This library is developed by PJRC, it utilizes a DSP instruction set within the Arm Cortex M4 for audio processing capabilities. It compiles fine with Teensyduino.

@ivankravets
Copy link
Member

I got you. Thanks a lot! 👍

Please move this #759 (comment) to https://github.com/platformio/platform-teensy/issues

This repository is intended for PlatformIO Core.

ivankravets added a commit that referenced this issue Sep 9, 2016
* develop:
  Fix incorrect line order when converting from INO to CPP and pointer is used
  Fix unit test
  Notify about `version` field when creating library
  Add support for SparkFun Blynk Board
  Return valid exit code from ``plaformio test`` command
  Disable SSL Server-Name-Indication for Python < 2.7.9
  Version bump to 3.0.1 (issue #772)
  Disable temporary SSL for PlatformIO services // Resolve #772
  Version bump to 3.0.0 (issues #770, #766, #747, #730, #765, #640, #659, #742, #459, #542, #763, #759, #753, #757, #749, #748, #745, #519, #709, #743, #413, #498, #410, #740, #361, #414, #554, #732, #588, #475, #461, #101, #719, #721, #537, #415, #522, #289, #556, #570, #456, #617, #432, #408, #479, #667, #510)
  Fix menu height for  docs
  Fix issue with multiple archives when linking firmware
  Add migration guide for PIO2 to PIO3
  Search libraries by headers/includes with ``platformio lib search --header`` option
  Update pio run command examples
  Add Unit Testing Demo
  Update PIO Plus badge title and link
  Add PlatformIO Plus badge
  Add links to PlatformIO Plus
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants