search

YunoHost / package_linter

Linter for YunoHost applications packages
https://yunohost.org/#/packaging_apps
GNU Affero General Public License v3.0
17 stars 13 forks source link

Syntax error detected, but there doesn't seem to be a problem with the syntax #94

Closed eauchat closed 3 years ago

eauchat commented 3 years ago

Maybe the syntax I'm using is not right, if so tell me, but I think it's fine, and testing it everything works good. However this test is displaying the following linter warning:

Some lines could not be parsed in script install. (That's probably not really critical)
? No closing quotation : couchdb couchdb/adminpass_again seen true" | debconf-set-selections

It seems that the linter doesn't like these lines:

echo "couchdb couchdb/mode select standalone
couchdb couchdb/mode seen true
couchdb couchdb/bindaddress string 127.0.0.1
couchdb couchdb/bindaddress seen true
couchdb couchdb/adminpass password ${COUCHDB_PASSWORD}
couchdb couchdb/adminpass seen true
couchdb couchdb/adminpass_again password ${COUCHDB_PASSWORD}
couchdb couchdb/adminpass_again seen true" | debconf-set-selections

Seems to me like a bug in the linter, but maybe I got something wrong..

alexAubin commented 3 years ago

It's a known issue in package_linter, it's not easy to properly parse stuff that involve multi-line strings

But it's no big deal because usually these are just strings and there's no actual packaging logic in those, so just ignore the message, like it says, "That's probably not really critical"