Skip to content

Commit

Permalink
Make --band mandatory. Update changelog.
Browse files Browse the repository at this point in the history
  • Loading branch information
brocaar committed May 8, 2016
1 parent adb54ca commit 25a54ae
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 4 deletions.
5 changes: 3 additions & 2 deletions cmd/loraserver/main.go
Original file line number Diff line number Diff line change
Expand Up @@ -42,6 +42,9 @@ func run(c *cli.Context) {
}

// get the band config
if c.String("band") == "" {
log.Fatalf("--band is undefined, valid options are: %s", strings.Join(bands, ", "))
}
bandConfig, err := band.GetConfig(band.Name(c.String("band")))
if err != nil {
log.Fatal(err)
Expand All @@ -51,7 +54,6 @@ func run(c *cli.Context) {
log.WithFields(log.Fields{
"version": version,
"net_id": netID.String(),
"nwk_id": netID.NwkID(),
"band": c.String("band"),
}).Info("starting LoRa Server")

Expand Down Expand Up @@ -168,7 +170,6 @@ func main() {
cli.StringFlag{
Name: "band",
Usage: fmt.Sprintf("ism band configuration to use. valid options are: %s", strings.Join(bands, ", ")),
Value: "EU_863_870",
EnvVar: "BAND",
},
cli.StringFlag{
Expand Down
4 changes: 2 additions & 2 deletions docs/changelog.md
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
# Changelog

## Untagged
## 0.6.1

* Band configuration can now be specified with the ``--band`` argument
* Band configuration must now be specified with the ``--band`` argument
(no more separate binaries per ism band)
* RX info notifications (``application/[AppEUI]/node/[DevEUI]/rxinfo``)

Expand Down

0 comments on commit 25a54ae

Please sign in to comment.