Yeah, it’s the QR code.

(function (i) { return function … })(i); inside the for loop. :-)

—C
(Sent from my iPhone)

> On 29 mai 2015, at 01:06, Ovidiu Cherecheș <notifications@github.com> wrote:
>
> You weren't supposed to find this
>
> —
> Reply to this email directly or view it on GitHub.
>


Hi guys,
we're in the middle of working on exactly these topics in Akka actually.
The most relevant tickets are:
- [Akka Serialization: manifest should be String, not Class based #17576](https://github.com/akka/akka/issues/17576) (has PR already)
- [Persistence "model adapter" for separating models and upcasting #17579](https://github.com/akka/akka/issues/17579) (currently drafting it)

If you could have a look and sanity check those features would address your needs that would be awesome.
Esp. we want to allow completely decoupling models - for example if a journal can work with BSON (we have another similar discussion running on a mongodb plugin), then an adapter should just be able to send BSON to the journal. In your case @ahjohannessen you could de-couple the domain classes from the "data representation model" in protobuf, and only send those to the journal.

Please have a look at the issues and let us know what you think!


please make a PR :)


will test and merge.. Thanks you so much....


Yes, tests need to initialize the permission checker if they invoke (directly or indirectly) any non-local service or any other code that does a permission check.

One way to do that is to apply the MainServletTestRule. @cgoncas can you take a look at the Test class just in case you have a better suggestion?


May be here directly?


Every get index will transform `join_models, fn(m) -> String.to_atom(m) end` more times. Why not transform somewhere before one time?


index = Enum.find_index(join_models, &(model == &1))


Thanks, that helped me  a lot!


You shouldn't do it like that.


Please describe what the bug was in commit messages... Easier to write the Changelog after


Ok, thanks. Just a change in config now.


thx @ktoso
@ahjohannessen so basically first bullet allows you to not bothering with PersistentRepr, EventStorePlugin could manage that on own level.


This will make tests hang as soon as it fails one day -_-


Seriously can't find a better way to test this??


model's scope CAN return any value, it will be casted to int. option's values MUST be int and function MUST return int


`join:my_model,my_model2` should work too?


@ktoso I already do that with scalaz bijections: domain <@> protobuf :)


makes sense and it's neat. thanks :)
+1 on renaming!


It seems we can. Nice catch, thanks. I'll fix this.


I wanted to add abstract interface, which would be used by EnterLeave, but there was some strange Lua error and I couldn't do anything.

I can add a special method which would emit mouse_enter, and call that Ceu object method from C++. Otherwise I'll have to pass C++ object like I do now.

And I've tried to make `el` a field of `MenuButton`, but got a message like it's "Not supported".


Принято


Sure, will do.

On Fri, May 29, 2015, 4:03 PM Achin Gupta <notifications@github.com> wrote:

> Hi Varun, Some coding style nits have been flagged by our CI. Could you at
> least fix the errors?
>
> Stacktrace
>
>   CHECKING STYLE
> WARNING: Do not use whitespace before Signed-off-by:
> #23:
>     Signed-off-by: Varun Wadekar <vwadekar@nvidia.com>
>
> WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
> #26:
> new file mode 100644
>
> WARNING: line over 80 characters
> #530: FILE: plat/nvidia/tegra/common/drivers/flowctrl/flowctrl.c:123:
> +   tegra_fc_write_32(FLOWCTRL_L2_FLUSH_CONTROL, !!MPIDR_AFFLVL1_VAL(mpidr));
>
> WARNING: line over 80 characters
> #550: FILE: plat/nvidia/tegra/common/drivers/flowctrl/flowctrl.c:143:
> +   tegra_fc_write_32(FLOWCTRL_L2_FLUSH_CONTROL, read_midr() == CORTEX_A53_MIDR);
>
> ERROR: do not use C99 // comments
> #635: FILE: plat/nvidia/tegra/common/drivers/flowctrl/flowctrl.c:228:
> +       ; // wait till value reaches EVP_BPMP_RESET_VECTOR
>
> ERROR: trailing statements should be on next line
> #640: FILE: plat/nvidia/tegra/common/drivers/flowctrl/flowctrl.c:233:
> +   while (val > mmio_read_32(TEGRA_TMRUS_BASE));
>
> WARNING: line over 80 characters
> #720: FILE: plat/nvidia/tegra/common/drivers/memctrl/memctrl.c:67:
> +   (void)tegra_mc_read_32(MC_SMMU_CONFIG_0);   /* read to flush writes */
>
> WARNING: line over 80 characters
> #724: FILE: plat/nvidia/tegra/common/drivers/memctrl/memctrl.c:71:
> +   tegra_mc_write_32(MC_SMMU_CONFIG_0, MC_SMMU_CONFIG_0_SMMU_ENABLE_ENABLE);
>
> WARNING: line over 80 characters
> #725: FILE: plat/nvidia/tegra/common/drivers/memctrl/memctrl.c:72:
> +   (void)tegra_mc_read_32(MC_SMMU_CONFIG_0);   /* read to flush writes */
>
> ERROR: do not use C99 // comments
> #866: FILE: plat/nvidia/tegra/common/drivers/pmc/pmc.c:116:
> +   reg |= 0x10;        // restart
>
> WARNING: externs should be avoided in .c files
> #929: FILE: plat/nvidia/tegra/common/tegra_bl31_setup.c:51:
> +extern unsigned long __RO_START__;
>
> WARNING: externs should be avoided in .c files
> #930: FILE: plat/nvidia/tegra/common/tegra_bl31_setup.c:52:
> +extern unsigned long __RO_END__;
>
> WARNING: externs should be avoided in .c files
> #931: FILE: plat/nvidia/tegra/common/tegra_bl31_setup.c:53:
> +extern unsigned long __BL31_END__;
>
> WARNING: externs should be avoided in .c files
> #934: FILE: plat/nvidia/tegra/common/tegra_bl31_setup.c:56:
> +extern unsigned long __COHERENT_RAM_START__;
>
> WARNING: externs should be avoided in .c files
> #935: FILE: plat/nvidia/tegra/common/tegra_bl31_setup.c:57:
> +extern unsigned long __COHERENT_RAM_END__;
>
> WARNING: externs should be avoided in .c files
> #938: FILE: plat/nvidia/tegra/common/tegra_bl31_setup.c:60:
> +extern uint64_t tegra_bl31_phys_base;
>
> WARNING: externs should be avoided in .c files
> #971: FILE: plat/nvidia/tegra/common/tegra_bl31_setup.c:93:
> +extern uint64_t ns_image_entrypoint;
>
> WARNING: line over 80 characters
> #1025: FILE: plat/nvidia/tegra/common/tegra_bl31_setup.c:147:
> +       plat_bl31_params_from_bl2.tzdram_size = plat_params->tzdram_size;
>
> WARNING: line over 80 characters
> #1026: FILE: plat/nvidia/tegra/common/tegra_bl31_setup.c:148:
> +       plat_bl31_params_from_bl2.bl32_params = plat_params->bl32_params;
>
> WARNING: externs should be avoided in .c files
> #1327: FILE: plat/nvidia/tegra/common/tegra_pm.c:46:
> +extern uint64_t tegra_bl31_phys_base;
>
> WARNING: externs should be avoided in .c files
> #1328: FILE: plat/nvidia/tegra/common/tegra_pm.c:47:
> +extern uint64_t sec_entry_point[PLATFORM_CORE_COUNT];
>
> WARNING: please, no space before tabs
> #1838: FILE: plat/nvidia/tegra/include/drivers/memctrl.h:50:
> +^I (TLB_FLUSH_VA_MATCH_ALL | ^I\$
>
> WARNING: line over 80 characters
> #1987: FILE: plat/nvidia/tegra/include/plat_macros.S:41:
> +   .asciz "gicd_ispendr regs (Offsets 0x200 - 0x278)\n Offset:\t\t\tvalue\n"
>
> WARNING: do not add new typedefs
> #2276: FILE: plat/nvidia/tegra/include/tegra_private.h:37:
> +typedef struct plat_params_from_bl2 {
>
> ERROR: do not use C99 // comments
> #2542: FILE: plat/nvidia/tegra/soc/t210/plat_secondary.c:40:
> +#define SB_AA64_RESET_LOW      0x30    // width = 31:0
>
> ERROR: do not use C99 // comments
> #2543: FILE: plat/nvidia/tegra/soc/t210/plat_secondary.c:41:
> +#define SB_AA64_RESET_HI       0x34    // width = 11:0
>
> WARNING: externs should be avoided in .c files
> #2545: FILE: plat/nvidia/tegra/soc/t210/plat_secondary.c:43:
> +extern void tegra_secure_entrypoint(void);
>
> WARNING: Do not use whitespace before Signed-off-by:
> #2708:
>     Signed-off-by: Varun Wadekar <vwadekar@nvidia.com>
>
> WARNING: Duplicate signature
> #2708:
>     Signed-off-by: Varun Wadekar <vwadekar@nvidia.com>
>
> total: 5 errors, 24 warnings, 2796 lines checked
>
> NOTE: Ignored message types: COMPLEX_MACRO GERRIT_CHANGE_ID GIT_COMMIT_ID
>
> Your patch has style problems, please review.
>
> If any of these errors are false positives, please report
> them to the maintainer, see CHECKPATCH in MAINTAINERS.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/vwadekar/arm-trusted-firmware/commit/7b6f4e81388c4b1cef50a675c4606a9a89fe72ea#commitcomment-11424572>
> .
>


The `(PFS)` seems a little incongruent here.  Maybe explain how it relates to the key exchange.


I actually used the same name as https://github.com/GoogleCloudPlatform/kubernetes/blob/677a4aa1a77147520845a191ef7f4a6df34a95f5/cmd/kubelet/app/server.go#L65


extract var for ```capConfig.zonename ? 'vm' : 'user'```


Why this change?
Seems unrelated to the commit.


There should be a blank line below.


it totally does.
shouldn't we also check per id, if the right docs were synced and not e.g. just duplicated?


Can we call this `template_namespace`?


else


This needs to be indented by four spaces in order for the markdown parser to wrap it in `<pre>` tags.


Can you create a branch with the Lua error?
Let me know the exact commit you are using (for `ceu` and `ceu-sdl`).


Very nice.Thanks man!


accepted


Can we use the full name instead of the aliases (`--app`, `--dir`)?


We generally discourage wildcard imports as per https://wiki.openmrs.org/display/docs/Coding+Conventions#CodingConventions-CodeStyle


UUID is not even close to being random...
http://tools.ietf.org/html/rfc4122#section-6


I think it's a little inconsistent that it returns either a populated object, an empty object or null.  I suggest to make it return null instead of an empty object.


嘛……Linux可以用mono跑gogotester找IP
我有个用python重写的gogotester可以用命令行跑求戳


Se ele vai ter o mesmo comportamento da tecla ENTER, não seria melhor assim?

```javascript
case KEYS.TAB:
case KEYS.ENTER:
  // ...
```

Se formos abrir um pull request (acredito que deveríamos), seria legal adicionar uma opção `tab_as_select: false`, pra garantir que o comportamento pode ser modificado de forma retrocompatível.


Style: `EVP_PKEY* key` (the star leans to the left.)


Hi Lubomir,

Where do smack.jar and smackx.jar come from ? The previous one contained important [fix] which appreantly is no longer present. Also What is the relation between smack and FFmpeg and G722 JNI libraries ?

Regards,
Pawel

[fix]: https://github.com/jitsi/libsrc/blob/master/smack/patches/15-remove-useless-room-listener-multiplexor-instance.patch


For consistency can we call this `Custom app name`?


`$format` is not expanded due to single quotation.


Thanks for the tip, I'll try to remember it. ;-)


@johnclause I was looking for the context of string "Slugs" and it's from a file that is missing: `/admin/qtx_admin_slug.php`
Is this a problem?


Removing the spaces is not following the WP coding standard :smile:


Hmm... I guess you're right. Damn.

29. května 2015 12:43:06 SELČ, "Jan Tvrdík" <notifications@github.com> napsal:
>:exclamation: :skull: :exclamation: :skull: :exclamation: :skull:
>:exclamation: :skull: :exclamation: :skull: :exclamation: :skull:
>:exclamation: :skull: :exclamation:
>WTF? Am I missing something or will this result in XSS when intuitively
>used, e.g. `{$comments->text|nl2br}`
>:exclamation: :skull: :exclamation: :skull: :exclamation: :skull:
>:exclamation: :skull: :exclamation: :skull: :exclamation: :skull:
>:exclamation: :skull: :exclamation:
>
>---
>Reply to this email directly or view it on GitHub:
>https://github.com/nette/latte/commit/86b55f5d8e7b0ed182404df987107e905a7f974f#commitcomment-11424716

--
Odesláno z mého telefonu s Androidem pomocí pošty K-9 Mail. Omluvte prosím moji stručnost.


I think `Use SHT instead of JST as template namespace` would be clearer


nice


Vamos criar um ```User#google_authorization``` para simplificar essa chamada?


also solange man keinen unterschied sieht sollte es egal sein. habs mal mit lossless gemacht so kann es keine beschwerden geben ;-)
Ok ich sehe man kann bei compressor.io lossy und lossless auswählen

hab jetzt nochmal mit meinem tool auf maximaler kompression versucht. da tut sich so gut wie nichts mehr...


`$format` is expanded before being placed in $output


No, we're fine here, because we're testing event handling here, not how api.sync() works. We already tested that in the other tests


"zur gleichen Qualität von ..." -> Ich glaube, darum geht es uns nicht. Wenn eduroam und restliches Netz beide kacke sind, ist das immer noch kacke. ;)

Vorschlag: "Resolution zur qualitativen Umsetzung von eduroam und anderen hoschulöffentlichen Netzwerken"

Thema Anführungszeichen: Ich dachte, in einem deutschsprachigen Text wären deutsche Anführungszeichen passend. Aber letztendlich ist das auch nur Kosmetik.


Perhaps make a mustParseKey function and use mustParseKey(string) here?


You are correct, my mistake.


something's missing here.
we set the logo's ratio on document ready. I have a small change to that part to avoid two issues. I wanted to push it to this branch but since that part misses...
https://github.com/Nikeo/customizr/blob/master/inc/assets/js/parts/main.js#L522

let me know, changes are silly if you want I can give you them here in plain text with a small comment.


You're now comparing the old foreground colour (which is likely to have been changed in the previous step if it would have made any difference) with the text background, which isn't quite what we want. Aside from anything else, changing the background colour to default makes the page look less ugly, so ideally the script would remove most background colours. I think it was better comparing the intensity of the post background against the text background.


That took me about 10 minutes to get to working earlier, I eventually got it working and just now remembered to shove it in somewhere. Mind merging branches?


This line is throwing an error for my on my own pfSense installation:
>Warning: implode(): Invalid arguments passed in /usr/local/pkg/pfblockerng/pfblockerng.inc on line 2639

[In master branch](https://github.com/pfsense/pfsense-packages/blob/master/config/pfblockerng/pfblockerng.inc#L2639)


What actual reason is there for them to be in the Oxide.Plugins namespace? There's no possible advantage to doing so, especially with Reflection being available.

If needed, I can share code I use in a production environment to compile and load C# on the fly (which includes type finders, and arbitrary namespaces + file/folder layouts). It'll take some tweaks since it's meant to be used for "real" .NET and attempts not to create/unload appdomains everywhere.


// TODO make it possible to make the user name site-specific.

?


> Kojak started off a is a collection of scripts

typo, or is it expected?


Any string that starts off initially beginning with a double-quote is allowed to have shell extensions and such inside of it. Any string that starts off initially beginning with a single quote takes every character literally, excluding probably a `\'`, I believe. This means that `"msg '$var'"` works because the `"` comes before the `'`. In the second case it doesn't work because it's declared a literal string.


// Set the Accept header to indicate that we're asking for
// a non-interactive login.


Unfortunately, when the output is empty, it displays nothing. Which it would currently be, due to missing feedback from the server.


`--silent`?


Sorry for pushing the help2man update to master. I wasn't are of the effects of this update. What is the best way to estimate such effects?


That's strange. Thought I removed that. I'm making a new commit now to fix.


May be split to to_lists and extract other logic to `Join` module?

```elixir
params = Enum.map(params, &to_list/1)
join_models = Exd.Builder.Join.models(params)
...
@lists ["full_join", "right_join", "left_join", "join", "load"]
defp to_list({key, value}) when key in @lists, do: {key, String.split(value, ",")}
defp to_list({key, value}), do: {key, value}
```


Ditto.


If c.Key isn't there, then surely agent login can't work.
Perhaps we should panic or return an error from agent.VisitWebPage
if c.Key is nil?


Ok dann vielleicht doch die Internetkompressoren?
Würde mich auch anbieten das zu machen. Lade es dann aber hier in einer .rar oder so hoch. Mit Git werde ich nicht warm, wie man gesehen hat.
Auch sollte man mal überdenken, ob man die Icons nicht überarbeitet. ~50% Einsparungen find ich nicht schlecht, auch wenn es vielleicht nicht bei jeder so viel sein wird.


@AruslanK firstly next time could you please use one of the following ways to ask us a question:

* [Issue Tracking](https://github.com/allure-framework/allure-core/issues?labels=&milestone=&page=1&state=open)
* Mailing list: [allure@yandex-team.ru](mailto:allure@yandex-team.ru)
* Gitter chat room: [https://gitter.im/allure-framework/allure-core](https://gitter.im/allure-framework/allure-core)
* StackOverflow tag: [Allure](http://stackoverflow.com/questions/tagged/allure)

I guess TestNG uses a different thread to notify listeners when listeners added via `testng.xml`. The same problem http://stackoverflow.com/questions/25717146/taking-screenshots-for-allure-reports-from-custom-testng-listener-testlistener


Thoughts on the name `keylen`?  Perhaps it could be more explicit about the fact that it's the length in bits, not bytes.


I'm quit sure this should have gone to Development.


Why not: if resp.StatusCode != http.StatusOK ?


Onun yerine if(file_exists) islemi yap else dosya yok de ve hatayi goster. Olumsuz kosul calismayacaksa ! kullanilir. Olumsuzu varsa once her zaman pozitif


thx!


Maybe rename `cb` to `next`?  Makes it a little clearer what its purpose is.


I am always interested to have a proper code!

However sometimes I feel that shellcheck is too picky, especially what double quoting is concerned as shellcheck isn't smart enough whether quoting is an issue or not..

Then there was a contribution and somebody ran it through shellcheck already && I did a bit after it.

Suggestion: 1) The code should be always that good that shellcheck won't mark anything in red -- at least if it's not a false positive. 2) Orange issues need further inspection. 3) Code changes then needs to be tested afterwards at least on BSD and Linux.


Thanks for pointingout gacl->get_object.  However, that function only returns the section_values and values, more work would be needed to get the displayable names with that mechanism.


Al sana odev bu degiskenlerin hepsini bir config.php dosyasina al. O degiskenleri burada kullan. config.php repo da olmasin. Ama app i alanin ne yapacagini gosteren bir config.sample.php dosyasi olsun. Read me de de izah et ne yapacagimizi

Dosyayi repodan kaldirmak icin

 git rm --cached <Dosya adi>


LGTM


Can you point me to your Admin/ACL code for reference?


bu commentler kod zaten versiyon kontrolu bunlar icin. Bu commentleri sil gereksiz.


s/of which key length is/with a key length/ and s/tls/TLS/


Bunlarida


Why (it->second).get rather than just it->second.get ?


!!!Indentation!!!


Ich verweise dann gleich mal auf dieses issue hier: #53
Wenn wir das umsetzen würden, würde es sowieso viel weniger icons geben

PS: Wenn du git hilfe braucht stehe ich gerne zur verfügung. Am anfang wird man etwas erdrückt von den 1000 möglichkeiten. Das hilft vll: https://git-scm.com/book/de/v1
Welches GIT Programm verwendest du? Ich bin zu 95% nur mehr in der Console unterwegs =)


I'm not entirely sure whether this is worth it.
In actual code, we'll want to be creating the httpbakery.Client
ourselves anyway, containing a custom cookie jar.

I think it's perhaps reasonable just to require clients to
set Key and VisitWebPage appropriately. I reserve the
right to change my mind as I look further down this
PR though :)


!!! Indentation !!!


Is Ca_t_ already accessible from the status dictionary? Or should it not be?


virgullerden sonra bir bosluk ve field name ler

 `id`, `firstname`, `lastname`

Seklinde olacak


Perhaps worth checking content type == json here, so we don't end up unmarshaling some apache response by accident?


Value null degil id yi cikar id field i auto increment et ve primary key olarak ata.

Mysql documentation a bak bunun icin


The package is named ```xlsx``` in the NPM registry, so previous line was correct (proof https://github.com/SheetJS/js-xlsx#installation)


Error ama napak :) hic bir sey yapmiyor bu :)


Hello, Pawel!

> Where do smack.jar and smackx.jar come from ?
Both smack.jar and smackx.jar come/are copied from jitsi/lib/installer-exclude/.

> The previous one contained important fix which appreantly is no longer present.
Thank you very much for the feedback! Are you saying that jicofo had it's own versions of smack.jar and smackx.jar separate from jitsi and all other projects dependent on these libraries? I could probably revert these two files for jicofo?

> Also What is the relation between smack and FFmpeg and G722 JNI libraries?
All files were modified as part of a split of third-party libraries licensed under the terms of the LGPL. FFmpeg and G722 were the most urgent, the commit comment was perhaps ill-chosen.

Best regards,
Lyubo Marinov


Why was this made a float here? Is it a stylistic thing?


!!! Indentation !!! 2 spaces for indentation no tabs.


Suggestion: add a check that if the options object has a `minDHkeylen` property, that it's a natural number (i.e. integral and greater than zero.)  A value that coerces to NaN would mean the check below always passes.


Shouldn't this be greater than or equal to in the string?


Can you s/cb/next/?


Just for the correctness, that's not my patch, it's from crashoverride not me
http://forum.odroid.com/viewtopic.php?p=78757#p78757


strictly speaking, this is no longer the identity manager now, but just some arbitrary discharge server that supports agent authentication.


!!! Indentation !!! bu feci :)


Neden id ekledin ?


I did it. Just check and pull will be sent.


Can you wrap this in braces?  And maybe split it into two statements for easier reading?


Again, why the parens around (x.y->z).w?


Ilk once commit mesajdan baslayalim kendisini aciklamiyor. Comentleri kaldiralim.


very useful comment, much appreciated thanks.


Esto no es correcto. ¿Por qué no lo has puesto como yo te he dicho?


@sergeybykov
> If we could also include #136 in 1.1.0, that would be a real milestone. :-)

Related to this, as a heads up, I think I'll do some work on the weekend on moving the code in https://github.com/dotnet/orleans/pull/347 towards "Orleans proper". I think I still keep them in a separate directory for now. I'll do this so that I can refactor the code the relational data access code and test them in the projects. If I remember correctly what I was thinking in #136 (and some other threads) was that it'd be nice if there something like ``Orleans.Storage`` project that would contain all the storage related code.

As usual, it's OK to chime in at any point if you have questions or other plans. :)


not sure if I get this pull request ... it works fine for me with ``context["request"]``. why do we need an update here?


The main reason of extraction was in my desire to keep all eggs in single basket: all assets-related work should be kept within single file or module. But that's just IMHO.


add space between ```=``` and method


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.196](http://teamcity.monogame.net/viewLog.html?buildId=7798&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


La ruta cambió a .local/share/nautilus/scripts y Mario la actualizó, en 14.04 el problema es que sale en el menú contextual de fichero y de carpeta, pero no sobre el fonde de escritorio ¿?  (pero sí si abrimos carpeta personal y ahí botón derecho encima de la carpeta escritorio)
 Lo voy mirando, prioridad baja.


What about `return mCriterion.emplace(...).second` ?


Do you really need to define this function?


The move badge works for me (using the old code)
![Moved thread badge](http://img.jackw.net/421432898888.png)
Once that badge is there, IMO it eliminates the confusion. When we move threads, we have the option not to leave the link in the original subforum, but we chose to leave the link so that people (who for the most most don't know how to use the My Content page, trust me - when we merge threads, it causes similar confusion). The thread doesn't say that it has 0 posts, though removing the 0 views counter could be something that is added to the script in future.
Personally, I use the moved thread links after just moving the thread to verify that it worked, though I know that it's a very niche use case.
As I said, feel fee to use it yourself, but it's not a feature that I would like to have in my version of this script.


I've tested this on the currently released version:

```bash
nscale 		 0.16.1
nscale-kernel 	 0.16.16
nscale-client 	 0.16.15
```

and on nscale@next:

```bash
nscale 		 0.17.0-pre4
nscale-kernel 	 0.17.1
nscale-client 	 0.17.2
```

when attempting to run

```bash
nscale container list b5168d74e74a (revision number on the sudc system)
unable to find system
```

and when running:

```bash
nscale container list sudc
Name                 Type                 Id
awsWebElb            aws-elb              awsWebElb
awsWebSg             aws-sg               awsWebSg
demo2                aws-ami              awsMachine
doc                  process              doc$77c4014bef47deb4fef3af579f2959457c058ce8
hist                 process              hist$39f0c71b89f3ba78064468c0af79017927f1a6cb
real                 process              real$5309ad7aeba319fd44adb18bbc983f4587f16af9
web                  process              web$e2021682ad1d25287321a4883535252ba684d9ba
awsAutoscaling       aws-autoscaling      awsAutoscaling
awsAMachine          blank-container      awsAMachine
root                 blank-container      root
```

@jsdevel Are can you confirm this on your machine for me? You might need to use a different revision number. Pick a random one from the output of `nscale rev list sudc`


No way lol, if i upgrade Dialer, Contacts and other telphony stuff i also need to update all telephony/net code in fwbase, it's hell...


ditto


For small loops/if, do not use an extra { }, directly put the single statement under the loop


>Thank you very much for the feedback! Are you saying that jicofo had its own versions of smack.jar and >smackx.jar separate from jitsi and all other projects dependent on these libraries? I could probably revert >these two files for jicofo?

Yes, I was too lazy to commit new version to all projects as they do not use that change. Are there any changes to smack jar files related to your work on LGPL ? I can't figure this out by looking at last commits in Jitsi source tree. If there aren't any then yes, please revert those two files. Thanks !


This may read more clearly in the usual multiline format.


A similar comment for this endpoint and the others would also make it easier to see the intent of the code.


orada deneme yanılma metodu şekli oldu biraz , gereksiz mi ?


Hi Nyall, this commit fixes http://hub.qgis.org/issues/12578 ? cheers!


Do you need this function? I'd think that shared_ptr already has this defined.


This won't work with the new registerCommand(...), it takes only command without name.


:joy:


Gereksiz


This feels potentially racy but the race detector likes it enough, so I'm not going to say any more.


нажата ли кнопка?


Ich schau mir  #53 mal an und auch die Programme dazu
Ich werde gerne darauf zurückkommen wegen Git. Noch "nutze" ich das Git was man hier bei Github bekommt.
Da meine Programmierfähigkeit aber gegen null geht, wird es bei mir bei so Sachen wie Icons anpassen bleiben.
Da ich momentan aber an meiner Projektarbeit sitze wird es mit #53 eher gegen Ende des nächsten Monats werden, bis ich dazu komme. Die Icons könnte man aber recht schnell anpassen.


One thing left out here is where and how you would like feedback or where the conversation continues.

I assume that you would want to point people to a wiki page allowing people to sign up and or suggest their thoughts on this.. or start a google group or hipchat or slack to continue the conversation.

At any rate, I am happy to help and still very NewB and easily confused by the new express-generator template, so I can hopefully give good feedback about what makes sense from my perspective.

That all said, I would prefer the branch model. One of the best parts of your book is that you are trying to get people to think in terms of real work projects and using branches seems to be more of what you would use so... it's more like a real work choice rather than one based on making things simple.

so that's my 2cents


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.196](http://teamcity.monogame.net/viewLog.html?buildId=7798&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:06:05


`$output` non-existant


4th commit


indent


I'll fix that now #153 is merged


Asana... that is where I would continue this discussion... if you build a team there is a team chat and then that can lead to tasks.. ;)


How about test := r.Form.Get("test")
and then using that in all the places you've written out r.Form.Get("test") below?


:D Could you instead summarize what we're asserting?


1: M111 unrecognized component name: base


@phpHavok Cool! Thanks for the contribution!


Since getValue is copying the value anyway, I'm not sure you gain anything here, but it is confusing to have two almost identical names declared next to each other, and the second one is in a scope that doesn't reference it.


Why not using the `pear` service? It is intended for that


Apparently, yes. At http://doc.qt.io/qt-5/qhash.html you see that there is no definition for shared_ptr, only for normal pointers T*.


Should be able to call getValue directly on i->second


'completionreserved' field should be listed in 'backup_reservation_stepslib.php' as per documentation:
https://docs.moodle.org/dev/Activity_completion_API#Backup_and_restore_for_completion_fields

Otherwise it will not be persisted and restored...


Or better, perhaps:
   sendResponse := func(err error) {
       d := i.discharges[r.Form.Get("test")]
       d.c <- err
       if err != nil {
            i.error(w, http.StatusInternalServerError, "%s", err.Error())
       }
   }
?


those indentations should be kept


checkers.ParseCaveat?


Would be great to also have the MTU option :) We use this in FFRL for providing lower MTU's than 1500 since our tunnels to the backbone only use 1400


It does not compile. I think i need the previous patch for that.


Stop messing with the colors and get the site up so I can link to you ;-)


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.197](http://teamcity.monogame.net/viewLog.html?buildId=7802&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


                                                                                Can you send me a PM in the pfSense forum?                                                                                From: MarkVLKSent: Friday, May 29, 2015 7:14 AMTo: pfsense/pfsense-packagesReply To: pfsense/pfsense-packagesCc: BBcan177Subject: Re: [pfsense-packages] pfBlockerNG v1.09 (e80da3c)This line is throwing an error for my on my own pfSense installation:

Warning: implode(): Invalid arguments passed in /usr/local/pkg/pfblockerng/pfblockerng.inc on line 2639

In master branch

—Reply to this email directly or view it on GitHub.


1.源代码管理器
2.设计文档
① 变量命名
② Visio画图
{
   ① 流程图 ： 系统大致如何奔跑
   ② 时序图 ： 发包按照时间的穷举，包括数据报包的类型
   ③ 效果图 ： 那些东西是输出，那些东西是输入，交互内容，应用服务器跑什么应用
}
③ 选择操作系统 Linux 或者是Windows
④ 开发工具（前提选择 什么语言）
⑤ 数据包
{                                         {
    控制包                                   数据包的字段
    数据包                                   首部                    控制字段
}                                          }

⑥  封包次序
⑦  每个包收到或发出要有日志文件（如何输出，带有时间和包内容的日志）


typo?


Could you implement this? It should be straight forward.


I'd have used
`enum class Type : signed char {
      INVALID = -1, ...`


typo


bu table adı ve değişkenleri unutmamak için böle yapmıştım :)


> Are there any changes to smack jar files related to your work on LGPL ?

I haven't modified or rebuilt either smack.jar or smackx.jar, I copied them from the jitsi project to all projects among jicofo, jigasi, jipopro, jirecon, jitsi-hammer, and jitsi-videobridge which depend on it.

> If there aren't any then yes, please revert those two files.

OK. (I don't like it because that's going to put jicofo in a unique situation with respect to the other projects and I find that to be a maintenance hardship. Anyway, I appear to have broken your work so I'm terribly sorry and I'll comply.)

By the way, how does it show that these versions are not right? I'm asking because I tested and jicofo appeared to work for me so I'm worried now that my testing was flawed. Thank you!


check -> isInSearch


Should be cleaner now, we have a BreakID per while too.
Mutation in the BodyTranslator with translatingWhile is not ideal, but I am not sure creating another environment for `while` would work just fine or not.


Nice - thanks for that.

We use git-flow so we only merge into the development branch, but the change is easy to implement, so will do it manually

Thanks again


No, contrary to `configDir`, the value of `rootDir` should never be `null`. To be on the extra safe side, https://github.com/sbesson/bioformats/commit/4396a1e3a2bf3a4c8dd2389f298663cae0c4a215 adds a check in the constructor.


Not sure this security checking is right. Looks like a person would need both Administrate authorization to the block and Edit authorization to calendar in order to edit it, when they should only need one or the other ( Administrate rights to block should give user ability to edit all data ).  Also, if you can, try to avoid having to call the same IsUserAuthorized more than once. Lines 342 and 353 both call `IsUserAuthorized ( Authorization.ADMINISTRATE )`. Call it once and set a flag. ( fyi, only noticed these because this section is getting a compiler warning about readOnly being set but never used ).


You want to make a pull request for this?


Yeah this was a mistake. My Sublime Text was removing all trailing white spaces.

 I reverted this in the next commit.


TeamCity xBIM :: XbimWebUI :: Build [Build 59](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1772&buildTypeId=XBIM_XbimWebUI_Build) is now running


Don't we need `Trill* Trill::findTrill(Chord* chord)` here? (Also note the position of the asterisk)


Hey chris, welcome to germany!!!! , I try to write in English, I need
practice a little bit... hahaha

i hope you to rest well and your "Jetlage" is disappearing fast!!!!

thanks you very much for you work in our website, really really thanks....
Because of the events in the website, i try to make good the things, but
maybe you can show me  how to make it better.
I dont want to disturb always when have a new event concert to website. I
now to that is no problem for you, but is your work and your time. I have
much more dates to put in the website, could we do it together one day?

Good mein Freund!!!!
see you soon!
 A lot of hugs

Tatan

2015-05-28 0:38 GMT+02:00 Chris Ruppel <notifications@github.com>:

> @tatani <https://github.com/tatani> man muss kein platz zwischen
> Dateinamen wörter stellen. Doof, weiß ich [image: :frowning:]
>
> Nutz minus anstatt nächstes mal. Ich repariere es jetzt (vom Flugzeug! [image:
> :airplane:] [image: :computer:] )
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/rupl/el-flecha-negra/commit/eaa686bb76315833c3bd9d7eb4bb67806832eb11#commitcomment-11396295>
> .
>

--

*Sebastián González LuisKomturstraße 3079106 Freiburg im
BreisgauDeutschland*


TeamCity xBIM :: XbimWebUI :: Build [Build 59](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1772&buildTypeId=XBIM_XbimWebUI_Build) outcome was **SUCCESS**
Summary: Running Build time: 00:00:30


TeamCity xBIM :: XbimWebUI :: Build [Build 60](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1773&buildTypeId=XBIM_XbimWebUI_Build) is now running


TeamCity xBIM :: XbimWebUI :: Build [Build 60](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1773&buildTypeId=XBIM_XbimWebUI_Build) outcome was **SUCCESS**
Summary: Running Build time: 00:00:29


There is the `maintainers/scripts/rebuild-amount.sh` script in Nixpkgs. I'm not sure how up-to-date it is. Also, `nox` seems to have a way of figuring out all affected packages.


We had a problem with handlers bound on body and listening on bubbled events from form that were not executed from older IE's.


That seems fragile -- why not just install `libgoogle-perftools-dev` which pulls `libtcmalloc-minimal4` as dependency and install SO symlink for you?


Yes, I'm Latvian. ;) Greetings to Lithuania! gj in eurovision! :+1: :D

I forked your project because I really love my debugging tool https://github.com/andrejsstepanovs/debug functionality that I made looong time ago and did not intended to support. After I stumbled on kint, I realized that it is providing Exactly the same functionality.

After using it for a view hours I realized that I can totally switch and use stable and maintained code.
Only thing that is missing for me is this logging functionality. So I quickly forked project, implemented what I need, and future plan was to polish my changes and make a merge request.

Your solution is elegant, but without new decorator it is too straight forward. Log entries are too huge and do not have time (and I need time).

Overall, give me time, I will clean up code in decorator and will improve newly created l() method. Will that be ok to accept future merge request?


**[unitTests_js_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_js_branches/12757/testReport/) for this commit**


Never heard about Gogs. Looks cool, thanks. :)
Maybe some comparison table (like the one on Wikipedia) would be better than a simple list…


Never mind, I'm looking into wrong lines... All good.


Interesting.  I'm running `nscale-client@0.16.15`, and get the reverse.  I am trying to hack on a new system though, so I wonder if I'm missing something.


Hmm, interesting. I'm not sure if there isn't some magic that makes it work. Did you try it?


Here I don't parse the users coarc and global settings ? I do parse those (check below)


**WHO ARE YOU!**


Looks good to me  :+1:


But the change is not seen in the function. Inside the function, it still checks the old constants


cool, thanks :+1:


To jest to o kobiecie przecietej w poł na moście i łysym policjancie i blond pani detektyw z drugiej storny granicy?


Do we have to keep /* ... */ here?


cool stuff, keeping each test muxer isolated to the specific test


ah but you don't merge them together


Sure. I've also two other patches, let's include them.


Its better to have them aligned.


This line breaks compiling for me. I have no idea why.

gcc version is: 4.8.4

Checkout my last commit which works for me: 7a1acbb483985f5a829dc867930ca9b719c1d5ca


So all I have to do is provide my own JSON parser in global space and it will work again in older IE?


I add a test class : h2/src/test/org/h2/samples/TestFunctionTrunc.java


nevajag


@wkennington This commit results in GCE instances that cannot be booted anymore. Can we revert this for now?


Unfortunately, this is not correct if you're calling the ``jetToolbox`` function multiple time for ``ak4`` jets but a different PU method (like CHS and puppi). Indeed, in this case, ``pileupJetIdEvaluator.jets`` and ``pileupJetIdCalculator.jets`` are set to the latest collection of jets added, and there's a crash when running cmsRun:

    ValueMap: no associated value for given product and index

The only fix is to load ``RecoJets.JetProducers.pileupjetidproducer_cfi`` with a postfix, like here: https://github.com/blinkseb/JMEValidator/blob/refactoring_74x_next/python/FrameworkConfiguration.py#L155

This means that PU jet ID calculators will run on each jets collections.


Looks like you're not supplying a valid unit type here for `background-size`?

Something like `.at2x('/media/img/firefox/personal/logo.png', 148px);` should work. Then you shouldn't need the line 103 above.


Yes, I tried.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.197](http://teamcity.monogame.net/viewLog.html?buildId=7802&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 486, ignored: 6 Build time: 00:20:37


I won't put the session.commit() in insertGroup(), e.g in this case I only want to commit once all fixtures are persisted.


yeah it's saw,
[commented on Github web client]


Everything is open source only have to mention also the developer who created the code. We all have a hobby and should not always argue " this is mine ". That brings us all not prevail.


shouldn't it be groupId ?


Can we remove this line?


Why did you remove that stroke type?
Afaik the type after the slash references the stroke in the original version, and the third stroke of 竹 *is* ㇑, not ㇔.


Again here I think `.at2x('/media/img/firefox/personal/mission-icons-sprite.png', 175px);` should be what you need? Then line 170 can be removed.


fix a typo lables->labels please


@HRTKernel It's common practice to simple cherry-pick the commits, though, leaving the author fully in-tact.


PKG_LICENSE missing, please add.


Well I agree, that current name is not the best but the suggested is not accure either. Because this save method is used when creating and editing pipelines. So in fact, this permission prevents user without the permission
 - create pipeline with PUBLIC_RW visibility
 - change visibility to PUBLIC_RW from PRIVATE / PUBLIC_RO
 - and finally, yes, also editing PUBLIC_RW pipelines
In fact, user without this role can edit PUBLIC_RW pipeline, but he cannot save it as PUBLIC_RW pipeline. If he changed it to PUBLIC_RO, he could save it (but I am not sure if this is permitted by UV as PUBLIC_RO is more restrictive). So if you consider this, the name of the permission is not so off.
But as I said, this is eDemo feature and standard UV users are not affected by this at all


See sql.replace("%", "/%") in memberSearchToSql


I think that use steps like:
the image should contain label {label} with value {value}
will be more readable


that's a typo. correct sentence is:
Kojak started off as a collection of scripts


I'm not sure I understand what you are saying. You think `init` is called on every frame?


also i think contain $s value is a typo too


Pas ok avec cette modif, pour afficher la description dans le modèle PDF expédition c'est dans le fichier pdf_rouget qu'il faut modifier l'appel à pdf_writelinedesc($pdf,$object,$i,$outputlangs,$this->posxqtyordered-10,3,$this->posxdesc,$curY,0,1); pour avoir $hidedetails et $hidedesc comme paramètre au lieu de 0,1 en derniers paramètres.


The check with type doesn't work for me. The if condition is true even though I check for python_foo for example.
Changing to:
if [ x$python_installed != x ] ; then

seems to work, but the type command still produces some output:
foo.sh: line 1: type: python_foo: not found

which is silent ;-) :
python_installed=`which python`


Closed Visual Studio Online Task [vso#4345](https://azurecom.visualstudio.com/DefaultCollection/ACOM/_workitems#_a=edit&id=4345): 'This is a test usersnap task submission'

VSO State: Done


is this really meant to be an instance variable? I don't see it being used anywhere else


It's right. Lerning by doing ; -)


``setOptional`` should mark it as defined, not as required


OK, we can do that I suppose (though I've always disliked projects that require extensive setup.)  I will mention, though, that when we create a full release, we need to have a full installer so the user can install it with absolutely no mess.


please fix some typos and consider step naming, thanks for PR!


I don't understand why this change is needed - could you explain? If you just want to wait for the child to exit, couldn't  you just just do `kill(pid, SIGTERM);` and then `waitpid(pid, &n, 0);`?

(I'm on rather shaky ground reviewing this pull request on the whole...)


It'd be nice to split this into two cases, so if `-e` isn't specified it still checks that there's exactly two arguments.


It is removed in a later commit: https://github.com/pontusmelke/neo4j/commit/0d1402801fdc7bad556748f84a1d2a91a8b24b36#diff-cb293639de5056ebf5273ce46ed25fe7L46
I think that is sensible enough.


This commit breaks the build here.
You should remove the ```BULLET_DYNAMICS_LIBRARY``` token line 69, then it works fine.


Hm, I may not have completely understood how this `kvg:type` thingy works. Can you please shed some light on it for me?

I guess this comment would apply to the second stroke below: should we keep the original `㇚`?


is not cpu_conf.h always included by cpu.h?


@jgm
>The one thing users may notice is different is that you can
no longer get the reference.docx or reference.odt using
`--print-default-data-file`.

It looks like one can still get the reference.docx with the `--print-default-data-file` (in 1.14.01).
(I don't think it is bad)


For instance, I wanted to use a commit that @temasek had on his repo, but he had cherry-picked from elsewhere. I cherry-picked his commit, which kept the original author in-tact like so DCTeam/android_frameworks_base@657ce470d8cb73b7cafd000a7446077224f25df3


Is the switch to using `sigaction` instead of `signal` just because it's the recommended practice nowadays, or is there another reason for it? (I'd probably make that change to the original a commit on its own if it's worth explaining.)


looks cool, thanks :+1:


Quick spell check on line 5 for 'themse'. Other than that great journal sir. I'm still struggling with committing early and often based on some ridiculous fear that my work isn't right and someone will look in my repo. :+1:


>OK. (I don't like it because that's going to put jicofo in a unique situation with respect to the other projects >and I find that to be a maintenance hardship. Anyway, I appear to have broken your work so I'm terribly >sorry and I'll comply.)

Yes, I also don't like it. I will push the change to all repos and propagate new version across the projects.

Btw. the real problems lies in the way we handle our project dependencies. Binary jar files are copied everywhere. I had this problem many times and it's a real pain. Maybe we should come up with something better. @ibauersachs  has propsed something about project dependency, but I don't remember the details right now - will have a look at that later.

>By the way, how does it show that these versions are not right? I'm asking because I tested and jicofo >appeared to work for me so I'm worried now that my testing was flawed. Thank you!

Yes, it will work, because the bug that has been fixed is hard to reproduce and it happens only when XMPP connection is broken. Recently IntelliJIDEA has added very usefull feature that allows to look up jar files and compare classes inside of them. It even decompiles Java byte code and you can see which lines are different ! :)


While uasset files can be big, if we don't add them to the repo how will we all have the same assets? We would each have to import all the resources into Unreal, and if we use different import settings it could create issues...


Yes, thanks for noticing. Fix: ff8fa3142c01f46e71578691c30b7b81fb109661


I might be completely misunderstanding, but as a simpler alternative, couldn't you use `setsid` in the child and then kill the process group?

(I guess the `exec` behaviour might be useful anyway to avoid shell quoting issues, though.)


@makcumm ?


curious what the use of isInEditMode() here and on line 389 is accomplishing?


you can delete this too


:v u clearly stole my script!


+1 for the autoconf-ery!


delete format and put directly 'json' in the last part of the URL


this is the old syntax already. You should use:
```
{{#each items as |content|}}
```


> Btw. the real problems lies in the way we handle our project dependencies. Binary jar files are copied everywhere. I had this problem many times and it's a real pain. Maybe we should come up with something better. @ibauersachs has propsed something about project dependency, but I don't remember the details right now - will have a look at that later.

I agree with you and I'm hoping to revive the discussion very soon. (I started reading the existing e-mail threads about Maven yesterday.)


is there a corresponding set of dimensions for various screen sizes (Ie in the hdpi folder)? Or does this look good across screen sizes?


> Hm, I may not have completely understood how this kvg:type thingy works.

[You are not alone.](https://groups.google.com/forum/#!topicsearchin/kanjivg/stroke$20types/kanjivg/8FQqN1wZl4s)

But i stay by my guess about the slash(stroke). It fits here. So i think the /㇑ and /㇚ should not be deleted.

And i have commented on the other changes to this kanji at #79.


This should be inside the try-catch block, otherwise if trigger checkpoint is called during poll it will kill the task with the uncaught interrupted exception


gpl v2


The patch seems sensible regardless, I was just making another suggestion.


Unnecessary: in SliceToPath(), you already check that the suffix is ".slice"


this one can be a param too


That's right , yes. He 's just a young developer and all will soon enough learn.


Better names for v and vg would be good (mView and mViewGroup probably)


this is so much cleaner.... great job


Nice to meet you, brāliukas :) Did Lithuania do well in Eurovision? Hm, good to know :)

Anyway, the decorator is created after **years** of trial and error finding the right readability that would work in real world debugging. I agree the title is a little large for logs, but if you would be dumping more than one variable at once, it becomes very unclear when one var dump ends and another begins - especially when dealing with text blocks. That's the reasoning it's so big, however I can see where you're comming from and sometimes the title block is superfluous and I sometimes don't want it either. I'll take a note of this as a potential improvement in the future (ability to switch off the huge header on the fly), but for now I'd much rather go with wrapping Kint instead of altering it.

Anyway, a new decorator would only be acceptable if it introduced a **radically** different display, far beyond making any of the available ones parameterizable.

Again, the improved function would look like this and can go anywhere in your project - it residing in `Kint.class.php` is not required at all.

```php
	/**
	 * Logs variable information to file; accepts arbitrary number of parameters
	 *
	 * [!!!] this function uses undocummented Kint configuration settings that
	 *       may become deprecated with future updates
	 */
	function l()
	{
		if ( !Kint::enabled() ) return;

		if ( !in_array( Kint::$aliases['functions'], 'l' ) ) {
			Kint::$aliases['functions'][] = 'l';
		}

		// adjust Kint settings manually, so this function accepts arbitrary number of
		// parameters as we can't use call_user_func_array with Kint's inline modifiers
		$enabled    = Kint::enabled( Kint::MODE_WHITESPACE );
		$outputMode = Kint::$returnOutput;
		Kint::$returnOutput = true;

		$params = func_get_args();
		$output = call_user_func_array( [ 'Kint', 'dump' ], $params );

		file_put_contents( 'log.txt', date( 'H:i:s' ) . PHP_EOL . $output . PHP_EOL, FILE_APPEND );

		Kint::enabled( $enabled );
		Kint::$returnOutput = $outputMode;
	}
```
it's not the *perfect* solution, as the variable names are still large and the date is not inline, but it supports passing multiple variables at once and just does the job.

On my end, I will make an announcement when I deprecate the configuration that this function uses with instructions on how to replace it - and will further look into optionally removing the big header.


It's because the child might ignore TERM, or might be hung -- TERM is just a 'please mr process will you exit', whereas KILL is handled in the kernel before the process even sees it.  Here `alarm_handler_kill` will send a KILL after `WAIT_AFTER_TERM` seconds (to give the process a chance to clean up and exit).


This seems to be wrong. If you clear the interrupted flag before the collect that means it might get interrupted again, so you will actually lose the element.

I might not get something though...


Yes I also tried to fix the tests. I think we should take the running testpart away from README. Can you add some of the examples that you though where wrong, which you mentioned in a comment in a issue. You did not specify what those cases where but since we have not fixed those I guess they still exist.


@KarishmaBagga, done: https://github.com/MSOpenTech/dash.js/commit/9e46248b72c10aa2c7114cab157c96eaeed4059e
I added a check for two cases:
1. if we reach the end of the file
2. if a segment is too big - I set a limit = 10000 bytes. Not sure that this is an optimal value. Any suggestions are welcome.


Add actual hostname


Thanks! I will check more carefully next time.


Two reasons: `signal()` is deprecated, and it doesn't let you set flags like `SA_NODEFER` (without which ARLM would be blocked while the signal handler is running).


I think the error message should be different. We are opening the credentials file here.


Indeed! I've removed them.


This will go away.


Cool :)


Thanks :)


@KozhinM Do we also need a check for isComplete here?


one more note, sorry for all the rambling... It would be useful if you gave each new file a name and explicitly labeled it within the book. For example on page 15 you are giving code that will be used to update the hello world file, but there is no box/title that tells you which file this should go into. Also, if you use branches it will be helpful to name all the example files, even if they are add ons to the orginal files (e.g. hello.js, hello-routing.js, hello-routing-static.js... or hello1.js, hello2.js... etc..)


there was not a repository we could use?
a repo would have the benefit to get you appropriate updates for the running platform, rather than having to hardcode a specific rpm.


comment hello


Fixed, thanks.


I think one has to be uncommented.


Could we avoid running these processes as root?


To be removed I guess, maybe used for testing?


Ah shoot, we have to add a different user status package to make this work? Do you think you could implement what tmeasday's package does in mizzao's package to add custom user states?


Perhaps just return the response as is if that happens?


For example:

def elevation_profile(request, format=None, path=None):
	if path:
		for latlng in path.split('|'):

Something like that?


Version 0.3.1 does not have the correct structure on Kodi 14.2


Let's have a test with an errorF returning an error with a cause here.


errgo.Mask(errorF(resp), errgo.Any)

would be more appropriate, I think, otherwise we make it impossible
for errorF to return an error with a cause.


1: M111 unrecognized component name: WebStyle


6: M102 unrecognized bullet/signature
7: M102 unrecognized bullet/signature
8: M102 unrecognized bullet/signature
1: M110 missing component name


3:1: E266 too many leading '#' for block comment
4:1: E266 too many leading '#' for block comment
6:1: E266 too many leading '#' for block comment
7:1: E266 too many leading '#' for block comment
8:1: E266 too many leading '#' for block comment
9:1: E266 too many leading '#' for block comment
11:1: E266 too many leading '#' for block comment
12:1: E266 too many leading '#' for block comment
13:1: E266 too many leading '#' for block comment
14:1: E266 too many leading '#' for block comment
16:1: E266 too many leading '#' for block comment
17:1: E266 too many leading '#' for block comment
18:1: E266 too many leading '#' for block comment
20: L101 copyright is missing
27:1: I101 Imported names are in the wrong order. Should be Cookie, add_cookie, get_cookie
29:1: I100 Imports statements are in the wrong order. import cPickle should be before from invenio.webinterface_handler_wsgi_utils
31:1: I100 Imports statements are in the wrong order. import random should be before import time
34:1: I100 Imports statements are in the wrong order. import os should be before import sys
40:1: I101 Imported names are in the wrong order. Should be blob_to_string, run_sql
40:1: E402 module level import not at top of file
41:1: I101 Imported names are in the wrong order. Should be CFG_WEBSESSION_EXPIRY_LIMIT_DEFAULT, CFG_WEBSESSION_EXPIRY_LIMIT_REMEMBER
41:1: I100 Imports statements are in the wrong order. from invenio.config should be before from invenio.dbquery
41:1: E402 module level import not at top of file
43:1: I101 Imported names are in the wrong order. Should be CFG_WEBSESSION_CLEANUP_CHANCE, CFG_WEBSESSION_COOKIE_NAME, CFG_WEBSESSION_ENABLE_LOCKING, CFG_WEBSESSION_ONE_DAY
43:1: E402 module level import not at top of file
47:1: E302 expected 2 blank lines, found 1
70:1: E302 expected 2 blank lines, found 1
70: D203 1 blank line required before class docstring (found 0)
90: D102 Missing docstring in public method
133:30: E261 at least two spaces before inline comment
163:17: E128 continuation line under-indented for visual indent
166:17: E128 continuation line under-indented for visual indent
169: D205 1 blank line required between summary line and description (found 0)
178:25: E127 continuation line over-indented for visual indent
205:22: E221 multiple spaces before operator
207:22: E221 multiple spaces before operator
212: D200 One-line docstring should fit on one line with quotes (found 3)
217:36: E203 whitespace before ':'
218:21: E128 continuation line under-indented for visual indent
218:31: E203 whitespace before ':'
219:21: E128 continuation line under-indented for visual indent
220:21: E128 continuation line under-indented for visual indent
220:31: E203 whitespace before ':'
221:21: E128 continuation line under-indented for visual indent
221:31: E203 whitespace before ':'
222:21: E128 continuation line under-indented for visual indent
222:32: E203 whitespace before ':'
223:21: E128 continuation line under-indented for visual indent
223:35: E203 whitespace before ':'
224:13: E124 closing bracket does not match visual indentation
245:17: E128 continuation line under-indented for visual indent
247: D200 One-line docstring should fit on one line with quotes (found 3)
254: D200 One-line docstring should fit on one line with quotes (found 3)
266: D400 First line should end with a period (not 'o')
266: D205 1 blank line required between summary line and description (found 0)
282: D400 First line should end with a period (not 's')
282: D205 1 blank line required between summary line and description (found 0)
292: D400 First line should end with a period (not 's')
292: D205 1 blank line required between summary line and description (found 0)
302: D200 One-line docstring should fit on one line with quotes (found 3)
309: D200 One-line docstring should fit on one line with quotes (found 3)
316: D205 1 blank line required between summary line and description (found 0)
323: D205 1 blank line required between summary line and description (found 0)
330: D205 1 blank line required between summary line and description (found 0)
337: D400 First line should end with a period (not 't')
337: D205 1 blank line required between summary line and description (found 0)
345: D400 First line should end with a period (not ',')
345: D205 1 blank line required between summary line and description (found 0)
353: D400 First line should end with a period (not ',')
353: D205 1 blank line required between summary line and description (found 0)
362: D200 One-line docstring should fit on one line with quotes (found 3)
366: D400 First line should end with a period (not 'd')
366: D205 1 blank line required between summary line and description (found 0)
378: D102 Missing docstring in public method
381:1: E302 expected 2 blank lines, found 1
381: D200 One-line docstring should fit on one line with quotes (found 3)
387:1: E302 expected 2 blank lines, found 1
387: D202 No blank lines allowed after function docstring (found 1)
387: D205 1 blank line required between summary line and description (found 0)
414:1: E302 expected 2 blank lines, found 1
414: D400 First line should end with a period (not 'r')
414: D205 1 blank line required between summary line and description (found 0)
431:1: E302 expected 2 blank lines, found 0
431: D205 1 blank line required between summary line and description (found 0)
450:1: E302 expected 2 blank lines, found 1
450: D202 No blank lines allowed after function docstring (found 1)
450: D400 First line should end with a period (not 'p')
450: D205 1 blank line required between summary line and description (found 0)


1: D400 First line should end with a period (not 's')
2:1: E266 too many leading '#' for block comment
3:1: E266 too many leading '#' for block comment
5:1: E266 too many leading '#' for block comment
6:1: E266 too many leading '#' for block comment
7:1: E266 too many leading '#' for block comment
8:1: E266 too many leading '#' for block comment
10:1: E266 too many leading '#' for block comment
11:1: E266 too many leading '#' for block comment
12:1: E266 too many leading '#' for block comment
13:1: E266 too many leading '#' for block comment
15:1: E266 too many leading '#' for block comment
16:1: E266 too many leading '#' for block comment
17:1: E266 too many leading '#' for block comment
19: L101 copyright is missing
28:2: E114 indentation is not a multiple of four (comment)
28:2: E116 unexpected indentation (comment)
29:2: E114 indentation is not a multiple of four (comment)
29:2: E116 unexpected indentation (comment)
30:2: E114 indentation is not a multiple of four (comment)
30:2: E116 unexpected indentation (comment)
31:2: E114 indentation is not a multiple of four (comment)
31:2: E116 unexpected indentation (comment)
32:2: E114 indentation is not a multiple of four (comment)
32:2: E116 unexpected indentation (comment)
33:2: E114 indentation is not a multiple of four (comment)
33:2: E116 unexpected indentation (comment)
34:2: E114 indentation is not a multiple of four (comment)
34:2: E116 unexpected indentation (comment)
35:2: E114 indentation is not a multiple of four (comment)
35:2: E116 unexpected indentation (comment)
36:2: E114 indentation is not a multiple of four (comment)
36:2: E116 unexpected indentation (comment)
37:2: E114 indentation is not a multiple of four (comment)
37:2: E116 unexpected indentation (comment)
38:2: E114 indentation is not a multiple of four (comment)
38:2: E116 unexpected indentation (comment)
39:2: E114 indentation is not a multiple of four (comment)
39:2: E116 unexpected indentation (comment)
40:2: E114 indentation is not a multiple of four (comment)
40:2: E116 unexpected indentation (comment)
41:2: E114 indentation is not a multiple of four (comment)
41:2: E116 unexpected indentation (comment)
42:2: E114 indentation is not a multiple of four (comment)
42:2: E116 unexpected indentation (comment)
43:2: E114 indentation is not a multiple of four (comment)
43:2: E116 unexpected indentation (comment)
44:2: E114 indentation is not a multiple of four (comment)
44:2: E116 unexpected indentation (comment)
48:1: B901 blind except: statement
51:1: I100 Imports statements are in the wrong order. import time should be before from wsgiref.headers
52:1: I100 Imports statements are in the wrong order. import re should be before import time
53:1: I100 Imports statements are in the wrong order. import os should be before import re
54:1: I100 Imports statements are in the wrong order. import cgi should be before import os
55:1: I100 Imports statements are in the wrong order. import cStringIO should be before import cgi
57:1: I101 Imported names are in the wrong order. Should be BuiltinFunctionType, ClassType, ListType, MethodType, TypeType
58:1: I201 Missing newline before sections or imports.
59:1: I101 Imported names are in the wrong order. Should be HTTP_BAD_REQUEST, HTTP_LENGTH_REQUIRED, InvenioWebInterfaceWSGIContentLenghtError, InvenioWebInterfaceWSGIContentMD5Error, InvenioWebInterfaceWSGIContentTypeError, SERVER_RETURN
71:1: E302 expected 2 blank lines, found 1
71: D101 Missing docstring in public class
74:5: E301 expected 1 blank line, found 0
74: D102 Missing docstring in public method
82:1: E266 too many leading '#' for block comment
83:1: E731 do not assign a lambda expression, use a def
85:1: E266 too many leading '#' for block comment
88: D101 Missing docstring in public class
90: D102 Missing docstring in public method
98:23: E202 whitespace before ')'
111:28: E721 do not compare types, use 'isinstance()'
117:37: W602 deprecated form of raising exception
135:1: E302 expected 2 blank lines, found 1
135: D203 1 blank line required before class docstring (found 0)
135: D400 First line should end with a period (not 't')
135: D205 1 blank line required between summary line and description (found 0)
148: D202 No blank lines allowed after function docstring (found 1)
148: D400 First line should end with a period (not 'n')
148: D205 1 blank line required between summary line and description (found 0)
160: D401 First line should be in imperative mood ('Thi', not 'This')
160: D201 No blank lines allowed before function docstring (found 1)
160: D400 First line should end with a period (not 'e')
160: D205 1 blank line required between summary line and description (found 0)
176:5: E303 too many blank lines (2)
176: D401 First line should be in imperative mood ('Provide', not 'Provides')
176: D201 No blank lines allowed before function docstring (found 1)
176: D202 No blank lines allowed after function docstring (found 1)
176: D400 First line should end with a period (not 'r')
176: D205 1 blank line required between summary line and description (found 0)
203: D102 Missing docstring in public method
205:33: E127 continuation line over-indented for visual indent
225:1: E302 expected 2 blank lines, found 1
243:1: E302 expected 2 blank lines, found 0
243: D401 First line should be in imperative mood ('Set', not 'Sets')
243: D202 No blank lines allowed after function docstring (found 1)
243: D400 First line should end with a period (not 'e')
243: D205 1 blank line required between summary line and description (found 0)
255:27: W601 .has_key() is deprecated, use 'in'
263:1: E302 expected 2 blank lines, found 1
263: D202 No blank lines allowed after function docstring (found 1)
263: D400 First line should end with a period (not 'n')
263: D205 1 blank line required between summary line and description (found 0)
270:26: W601 .has_key() is deprecated, use 'in'
274:22: E721 do not compare types, use 'isinstance()'
279:1: E302 expected 2 blank lines, found 1
279: D103 Missing docstring in public function
281:15: W601 .has_key() is deprecated, use 'in'
305:1: E302 expected 2 blank lines, found 1
305: D101 Missing docstring in public class
306: D102 Missing docstring in public method
326: D102 Missing docstring in public method
327:48: E251 unexpected spaces around keyword / parameter equals
327:50: E251 unexpected spaces around keyword / parameter equals
333:24: W601 .has_key() is deprecated, use 'in'
341:35: W604 backticks are deprecated, use 'repr()'
343: D102 Missing docstring in public method
345:33: W602 deprecated form of raising exception
354: D102 Missing docstring in public method
357:1: E302 expected 2 blank lines, found 1
357: D210 No whitespaces allowed surrounding docstring text
357: D203 1 blank line required before class docstring (found 0)
357: D204 1 blank line required after class docstring (found 0)
357: D400 First line should end with a period (not 'h')
357: D205 1 blank line required between summary line and description (found 0)
372: D300 Use """triple double quotes""" (found '''-quotes)
377: D102 Missing docstring in public method
379:33: W602 deprecated form of raising exception
385:35: W604 backticks are deprecated, use 'repr()'
387:1: E302 expected 2 blank lines, found 1
387: D101 Missing docstring in public class
389: D102 Missing docstring in public method
393: D102 Missing docstring in public method
403: D102 Missing docstring in public method
407: D102 Missing docstring in public method
411: D102 Missing docstring in public method
415: D102 Missing docstring in public method
419: D102 Missing docstring in public method
423: D102 Missing docstring in public method
427: D102 Missing docstring in public method
431: D102 Missing docstring in public method
435: D102 Missing docstring in public method
439: D102 Missing docstring in public method
443: D102 Missing docstring in public method
448: D101 Missing docstring in public class
450: D102 Missing docstring in public method
472:32: W602 deprecated form of raising exception
476:30: W601 .has_key() is deprecated, use 'in'
488:9: E303 too many blank lines (2)
502:32: W602 deprecated form of raising exception
508:52: E261 at least two spaces before inline comment
509:13: E266 too many leading '#' for block comment
529:36: E231 missing whitespace after ','
534:52: E201 whitespace after '['
574:28: W601 .has_key() is deprecated, use 'in'
581:28: W601 .has_key() is deprecated, use 'in'
599:28: W601 .has_key() is deprecated, use 'in'
613: D400 First line should end with a period (not 'r')
619: D102 Missing docstring in public method
621:17: W601 .has_key() is deprecated, use 'in'
629: D102 Missing docstring in public method
653:54: E701 multiple statements on one line (colon)
664:54: E701 multiple statements on one line (colon)
675:54: E701 multiple statements on one line (colon)
679: D102 Missing docstring in public method
690: D102 Missing docstring in public method
696: D402 First line should not be the function's "signature"
700: D102 Missing docstring in public method
703: D102 Missing docstring in public method
706: D402 First line should not be the function's "signature"
716: D210 No whitespaces allowed surrounding docstring text
716: D400 First line should end with a period (not 'd')
723: D210 No whitespaces allowed surrounding docstring text
723: D400 First line should end with a period (not 's')
730: D402 First line should not be the function's "signature"
730: D209 Multi-line docstring closing quotes should be on a separate line
730: D400 First line should end with a period (not 'e')
730: D205 1 blank line required between summary line and description (found 0)
735: D102 Missing docstring in public method
741: D102 Missing docstring in public method
745: D202 No blank lines allowed after function docstring (found 1)
766:1: E302 expected 2 blank lines, found 1
766: D202 No blank lines allowed after function docstring (found 1)
766: D400 First line should end with a period (not 'e')
766: D205 1 blank line required between summary line and description (found 0)
787:35: E231 missing whitespace after ','
815:5: E129 visually indented line with same indent as next logical line
830:1: E302 expected 2 blank lines, found 0
830: D205 1 blank line required between summary line and description (found 0)
838:5: E266 too many leading '#' for block comment
847:5: E266 too many leading '#' for block comment
851:5: E266 too many leading '#' for block comment
856:5: E266 too many leading '#' for block comment
861:5: E266 too many leading '#' for block comment
864:5: E266 too many leading '#' for block comment
868:13: E266 too many leading '#' for block comment
869:13: E266 too many leading '#' for block comment
875:13: E266 too many leading '#' for block comment
877:9: E266 too many leading '#' for block comment
881:13: E266 too many leading '#' for block comment
884:9: E266 too many leading '#' for block comment
889:5: E266 too many leading '#' for block comment


3:1: E266 too many leading '#' for block comment
4:1: E266 too many leading '#' for block comment
6:1: E266 too many leading '#' for block comment
7:1: E266 too many leading '#' for block comment
8:1: E266 too many leading '#' for block comment
9:1: E266 too many leading '#' for block comment
11:1: E266 too many leading '#' for block comment
12:1: E266 too many leading '#' for block comment
13:1: E266 too many leading '#' for block comment
14:1: E266 too many leading '#' for block comment
16:1: E266 too many leading '#' for block comment
17:1: E266 too many leading '#' for block comment
18:1: E266 too many leading '#' for block comment
20: L101 copyright is missing
29:1: I100 Imports statements are in the wrong order. import cPickle should be before from invenio.webinterface_handler_wsgi_utils
31:1: I100 Imports statements are in the wrong order. import random should be before import time
34:1: I100 Imports statements are in the wrong order. import os should be before import sys
40:1: E402 module level import not at top of file
41:1: I101 Imported names are in the wrong order. Should be blob_to_string, run_sql
41:1: E402 module level import not at top of file
42:1: I101 Imported names are in the wrong order. Should be CFG_SITE_SECURE_URL, CFG_SITE_URL, CFG_WEBSEARCH_PREV_NEXT_HIT_FOR_GUESTS, CFG_WEBSESSION_EXPIRY_LIMIT_DEFAULT, CFG_WEBSESSION_EXPIRY_LIMIT_REMEMBER, CFG_WEBSESSION_IPADDR_CHECK_SKIP_BITS
42:1: I100 Imports statements are in the wrong order. from invenio.config should be before from invenio.dbquery
42:1: E402 module level import not at top of file
46:1: I101 Imported names are in the wrong order. Should be CFG_WEBSESSION_CLEANUP_CHANCE, CFG_WEBSESSION_COOKIE_NAME, CFG_WEBSESSION_ONE_DAY
46:1: E402 module level import not at top of file
56:1: E302 expected 2 blank lines, found 1
79:1: E302 expected 2 blank lines, found 1
79: D203 1 blank line required before class docstring (found 0)
99: D102 Missing docstring in public method
160:13: E266 too many leading '#' for block comment
161:13: E266 too many leading '#' for block comment
162:13: E266 too many leading '#' for block comment
166: D401 First line should be in imperative mood ('I', not 'Is')
166: D400 First line should end with a period (not '?')
166: D200 One-line docstring should fit on one line with quotes (found 3)
172: D400 First line should end with a period (not 'r')
172: D205 1 blank line required between summary line and description (found 0)
181: D102 Missing docstring in public method
186: D102 Missing docstring in public method
202:17: E128 continuation line under-indented for visual indent
205:17: E128 continuation line under-indented for visual indent
209: D205 1 blank line required between summary line and description (found 0)
218:25: E127 continuation line over-indented for visual indent
225:25: E266 too many leading '#' for block comment
231:29: E131 continuation line unaligned for hanging indent
239:25: E266 too many leading '#' for block comment
245:29: E131 continuation line unaligned for hanging indent
261:22: E221 multiple spaces before operator
263:22: E221 multiple spaces before operator
268: D400 First line should end with a period (not 'd')
268: D205 1 blank line required between summary line and description (found 0)
278: D200 One-line docstring should fit on one line with quotes (found 3)
284:13: E266 too many leading '#' for block comment
285:36: E203 whitespace before ':'
286:21: E128 continuation line under-indented for visual indent
286:31: E203 whitespace before ':'
287:21: E128 continuation line under-indented for visual indent
288:21: E128 continuation line under-indented for visual indent
288:31: E203 whitespace before ':'
289:21: E128 continuation line under-indented for visual indent
289:31: E203 whitespace before ':'
290:21: E128 continuation line under-indented for visual indent
290:32: E203 whitespace before ':'
291:21: E128 continuation line under-indented for visual indent
291:35: E203 whitespace before ':'
292:13: E124 closing bracket does not match visual indentation
313:17: E128 continuation line under-indented for visual indent
316:9: E266 too many leading '#' for block comment
319: D200 One-line docstring should fit on one line with quotes (found 3)
326: D200 One-line docstring should fit on one line with quotes (found 3)
339: D400 First line should end with a period (not 'g')
339: D205 1 blank line required between summary line and description (found 0)
366:5: E303 too many blank lines (2)
366: D400 First line should end with a period (not 's')
366: D205 1 blank line required between summary line and description (found 0)
376: D400 First line should end with a period (not 's')
376: D205 1 blank line required between summary line and description (found 0)
386: D205 1 blank line required between summary line and description (found 0)
393: D205 1 blank line required between summary line and description (found 0)
400: D205 1 blank line required between summary line and description (found 0)
407: D400 First line should end with a period (not 't')
407: D205 1 blank line required between summary line and description (found 0)
415: D400 First line should end with a period (not ',')
415: D205 1 blank line required between summary line and description (found 0)
423: D400 First line should end with a period (not ',')
423: D205 1 blank line required between summary line and description (found 0)
432: D200 One-line docstring should fit on one line with quotes (found 3)
436: D400 First line should end with a period (not 'd')
436: D205 1 blank line required between summary line and description (found 0)
448:5: E266 too many leading '#' for block comment
449:5: E266 too many leading '#' for block comment
450:5: E266 too many leading '#' for block comment
451:5: E266 too many leading '#' for block comment
455: D102 Missing docstring in public method
458:5: E266 too many leading '#' for block comment
459:5: E266 too many leading '#' for block comment
460:5: E266 too many leading '#' for block comment
461:5: E266 too many leading '#' for block comment
462:5: E266 too many leading '#' for block comment
463:5: E266 too many leading '#' for block comment
464:5: E266 too many leading '#' for block comment
465:5: E266 too many leading '#' for block comment
466:5: E266 too many leading '#' for block comment
469:1: E302 expected 2 blank lines, found 1
469: D202 No blank lines allowed after function docstring (found 1)
469: D205 1 blank line required between summary line and description (found 0)
496:1: E302 expected 2 blank lines, found 1
496: D400 First line should end with a period (not 'r')
496: D205 1 blank line required between summary line and description (found 0)
513:1: E302 expected 2 blank lines, found 0
513: D205 1 blank line required between summary line and description (found 0)
532:1: E302 expected 2 blank lines, found 1
532: D202 No blank lines allowed after function docstring (found 1)
532: D400 First line should end with a period (not 'p')
532: D205 1 blank line required between summary line and description (found 0)
570:1: E302 expected 2 blank lines, found 1
570: D400 First line should end with a period (not 'P')
570: D200 One-line docstring should fit on one line with quotes (found 3)
575:17: E211 whitespace before '('
575:32: E211 whitespace before '('


29:1: I100 Imports statements are in the wrong order. import cPickle should be before from invenio.webinterface_handler_wsgi_utils
31:1: I100 Imports statements are in the wrong order. import random should be before import time
34:1: I100 Imports statements are in the wrong order. import os should be before import sys
40:1: E402 module level import not at top of file
41:1: E402 module level import not at top of file
42:1: I101 Imported names are in the wrong order. Should be CFG_SITE_SECURE_URL, CFG_SITE_URL, CFG_WEBSEARCH_PREV_NEXT_HIT_FOR_GUESTS, CFG_WEBSESSION_EXPIRY_LIMIT_DEFAULT, CFG_WEBSESSION_EXPIRY_LIMIT_REMEMBER, CFG_WEBSESSION_IPADDR_CHECK_SKIP_BITS, CFG_WEBSESSION_STORAGE
42:1: I100 Imports statements are in the wrong order. from invenio.config should be before from invenio.dbquery
42:1: E402 module level import not at top of file
49:1: I101 Imported names are in the wrong order. Should be CFG_WEBSESSION_CLEANUP_CHANCE, CFG_WEBSESSION_COOKIE_NAME, CFG_WEBSESSION_ONE_DAY
49:1: E402 module level import not at top of file
50:39: E128 continuation line under-indented for visual indent
51:39: E128 continuation line under-indented for visual indent
52:1: I100 Imports statements are in the wrong order. from invenio.dbquery should be before from invenio.websession_config
52:1: E402 module level import not at top of file
53:1: E402 module level import not at top of file
63:1: E302 expected 2 blank lines, found 1
87: D203 1 blank line required before class docstring (found 0)
107: D102 Missing docstring in public method
169:13: E266 too many leading '#' for block comment
170:13: E266 too many leading '#' for block comment
171:13: E266 too many leading '#' for block comment
175: D401 First line should be in imperative mood ('I', not 'Is')
175: D400 First line should end with a period (not '?')
175: D200 One-line docstring should fit on one line with quotes (found 3)
181: D400 First line should end with a period (not 'r')
181: D205 1 blank line required between summary line and description (found 0)
190: D102 Missing docstring in public method
195: D102 Missing docstring in public method
211:17: E128 continuation line under-indented for visual indent
214:17: E128 continuation line under-indented for visual indent
218: D205 1 blank line required between summary line and description (found 0)
233:25: E266 too many leading '#' for block comment
239:29: E131 continuation line unaligned for hanging indent
247:25: E266 too many leading '#' for block comment
253:29: E131 continuation line unaligned for hanging indent
269:22: E221 multiple spaces before operator
271:22: E221 multiple spaces before operator
276: D400 First line should end with a period (not 'd')
276: D205 1 blank line required between summary line and description (found 0)
286: D200 One-line docstring should fit on one line with quotes (found 3)
292:13: E266 too many leading '#' for block comment
293:36: E203 whitespace before ':'
294:21: E128 continuation line under-indented for visual indent
294:31: E203 whitespace before ':'
295:21: E128 continuation line under-indented for visual indent
296:21: E128 continuation line under-indented for visual indent
296:31: E203 whitespace before ':'
297:21: E128 continuation line under-indented for visual indent
297:31: E203 whitespace before ':'
298:21: E128 continuation line under-indented for visual indent
298:32: E203 whitespace before ':'
299:21: E128 continuation line under-indented for visual indent
299:35: E203 whitespace before ':'
300:13: E124 closing bracket does not match visual indentation
310:9: E266 too many leading '#' for block comment
313: D200 One-line docstring should fit on one line with quotes (found 3)
320: D200 One-line docstring should fit on one line with quotes (found 3)
333: D400 First line should end with a period (not 'g')
333: D205 1 blank line required between summary line and description (found 0)
360:5: E303 too many blank lines (2)
360: D400 First line should end with a period (not 's')
360: D205 1 blank line required between summary line and description (found 0)
370: D400 First line should end with a period (not 's')
370: D205 1 blank line required between summary line and description (found 0)
380: D205 1 blank line required between summary line and description (found 0)
387: D205 1 blank line required between summary line and description (found 0)
394: D205 1 blank line required between summary line and description (found 0)
401: D400 First line should end with a period (not 't')
401: D205 1 blank line required between summary line and description (found 0)
409: D400 First line should end with a period (not ',')
409: D205 1 blank line required between summary line and description (found 0)
417: D400 First line should end with a period (not ',')
417: D205 1 blank line required between summary line and description (found 0)
426: D200 One-line docstring should fit on one line with quotes (found 3)
434:5: E266 too many leading '#' for block comment
435:5: E266 too many leading '#' for block comment
436:5: E266 too many leading '#' for block comment
437:5: E266 too many leading '#' for block comment
441: D102 Missing docstring in public method
444:5: E266 too many leading '#' for block comment
445:5: E266 too many leading '#' for block comment
446:5: E266 too many leading '#' for block comment
447:5: E266 too many leading '#' for block comment
448:5: E266 too many leading '#' for block comment
449:5: E266 too many leading '#' for block comment
450:5: E266 too many leading '#' for block comment
451:5: E266 too many leading '#' for block comment
452:5: E266 too many leading '#' for block comment
455:1: E302 expected 2 blank lines, found 1
455: D202 No blank lines allowed after function docstring (found 1)
455: D205 1 blank line required between summary line and description (found 0)
482:1: E302 expected 2 blank lines, found 1
482: D400 First line should end with a period (not 'r')
482: D205 1 blank line required between summary line and description (found 0)
499:1: E302 expected 2 blank lines, found 0
499: D205 1 blank line required between summary line and description (found 0)
518:1: E302 expected 2 blank lines, found 1
518: D202 No blank lines allowed after function docstring (found 1)
518: D400 First line should end with a period (not 'p')
518: D205 1 blank line required between summary line and description (found 0)
556:1: E302 expected 2 blank lines, found 1
556: D400 First line should end with a period (not 'P')
556: D200 One-line docstring should fit on one line with quotes (found 3)
567:1: E303 too many blank lines (3)
567: D101 Missing docstring in public class
568: D102 Missing docstring in public method
570: D400 First line should end with a period (not 'd')
570: D205 1 blank line required between summary line and description (found 0)
581: D102 Missing docstring in public method
587: D102 Missing docstring in public method
591: D102 Missing docstring in public method
607:13: E128 continuation line under-indented for visual indent
610: D101 Missing docstring in public class
612: D102 Missing docstring in public method
615: D102 Missing docstring in public method
618: D102 Missing docstring in public method
621: D102 Missing docstring in public method


1: D400 First line should end with a period (not 's')
28:2: E114 indentation is not a multiple of four (comment)
28:2: E116 unexpected indentation (comment)
29:2: E114 indentation is not a multiple of four (comment)
29:2: E116 unexpected indentation (comment)
30:2: E114 indentation is not a multiple of four (comment)
30:2: E116 unexpected indentation (comment)
31:2: E114 indentation is not a multiple of four (comment)
31:2: E116 unexpected indentation (comment)
32:2: E114 indentation is not a multiple of four (comment)
32:2: E116 unexpected indentation (comment)
33:2: E114 indentation is not a multiple of four (comment)
33:2: E116 unexpected indentation (comment)
34:2: E114 indentation is not a multiple of four (comment)
34:2: E116 unexpected indentation (comment)
35:2: E114 indentation is not a multiple of four (comment)
35:2: E116 unexpected indentation (comment)
36:2: E114 indentation is not a multiple of four (comment)
36:2: E116 unexpected indentation (comment)
37:2: E114 indentation is not a multiple of four (comment)
37:2: E116 unexpected indentation (comment)
38:2: E114 indentation is not a multiple of four (comment)
38:2: E116 unexpected indentation (comment)
39:2: E114 indentation is not a multiple of four (comment)
39:2: E116 unexpected indentation (comment)
40:2: E114 indentation is not a multiple of four (comment)
40:2: E116 unexpected indentation (comment)
41:2: E114 indentation is not a multiple of four (comment)
41:2: E116 unexpected indentation (comment)
42:2: E114 indentation is not a multiple of four (comment)
42:2: E116 unexpected indentation (comment)
43:2: E114 indentation is not a multiple of four (comment)
43:2: E116 unexpected indentation (comment)
44:2: E114 indentation is not a multiple of four (comment)
44:2: E116 unexpected indentation (comment)
47:1: I100 Imports statements are in the wrong order. import time should be before from wsgiref.headers
48:1: I100 Imports statements are in the wrong order. import re should be before import time
49:1: I100 Imports statements are in the wrong order. import os should be before import re
50:1: I100 Imports statements are in the wrong order. import cgi should be before import os
51:1: I100 Imports statements are in the wrong order. import cStringIO should be before import cgi
53:1: I101 Imported names are in the wrong order. Should be BuiltinFunctionType, ClassType, ListType, MethodType, TypeType
54:1: I201 Missing newline before sections or imports.
55:1: I101 Imported names are in the wrong order. Should be HTTP_BAD_REQUEST, HTTP_LENGTH_REQUIRED, InvenioWebInterfaceWSGIContentLenghtError, InvenioWebInterfaceWSGIContentMD5Error, InvenioWebInterfaceWSGIContentTypeError, SERVER_RETURN
63:1: E302 expected 2 blank lines, found 1
63: D101 Missing docstring in public class
66:5: E301 expected 1 blank line, found 0
66: D102 Missing docstring in public method
75:1: E731 do not assign a lambda expression, use a def
80: D101 Missing docstring in public class
82: D102 Missing docstring in public method
90:23: E202 whitespace before ')'
103:28: E721 do not compare types, use 'isinstance()'
109:37: W602 deprecated form of raising exception
127:1: E302 expected 2 blank lines, found 1
127: D203 1 blank line required before class docstring (found 0)
127: D400 First line should end with a period (not 't')
127: D205 1 blank line required between summary line and description (found 0)
140: D202 No blank lines allowed after function docstring (found 1)
140: D400 First line should end with a period (not 'n')
140: D205 1 blank line required between summary line and description (found 0)
152: D401 First line should be in imperative mood ('Thi', not 'This')
152: D201 No blank lines allowed before function docstring (found 1)
152: D400 First line should end with a period (not 'e')
152: D205 1 blank line required between summary line and description (found 0)
168:5: E303 too many blank lines (2)
168: D401 First line should be in imperative mood ('Provide', not 'Provides')
168: D201 No blank lines allowed before function docstring (found 1)
168: D202 No blank lines allowed after function docstring (found 1)
168: D400 First line should end with a period (not 'r')
168: D205 1 blank line required between summary line and description (found 0)
195: D102 Missing docstring in public method
197:33: E127 continuation line over-indented for visual indent
217:1: E302 expected 2 blank lines, found 1
235: D400 First line should end with a period (not 'n')
235: D205 1 blank line required between summary line and description (found 0)
241:8: E713 test for membership should be 'not in'
251: D103 Missing docstring in public function
277:1: E302 expected 2 blank lines, found 1
277: D101 Missing docstring in public class
278: D102 Missing docstring in public method
298: D102 Missing docstring in public method
299:48: E251 unexpected spaces around keyword / parameter equals
299:50: E251 unexpected spaces around keyword / parameter equals
305:24: W601 .has_key() is deprecated, use 'in'
313:35: W604 backticks are deprecated, use 'repr()'
315: D102 Missing docstring in public method
317:33: W602 deprecated form of raising exception
326: D102 Missing docstring in public method
329:1: E302 expected 2 blank lines, found 1
329: D210 No whitespaces allowed surrounding docstring text
329: D203 1 blank line required before class docstring (found 0)
329: D204 1 blank line required after class docstring (found 0)
329: D400 First line should end with a period (not 'h')
329: D205 1 blank line required between summary line and description (found 0)
342: D300 Use """triple double quotes""" (found '''-quotes)
348: D102 Missing docstring in public method
351: D102 Missing docstring in public method
353:33: W602 deprecated form of raising exception
359:35: W604 backticks are deprecated, use 'repr()'
361:1: E302 expected 2 blank lines, found 1
361: D101 Missing docstring in public class
363: D102 Missing docstring in public method
367: D102 Missing docstring in public method
377: D102 Missing docstring in public method
381: D102 Missing docstring in public method
385: D102 Missing docstring in public method
389: D102 Missing docstring in public method
393: D102 Missing docstring in public method
397: D102 Missing docstring in public method
401: D102 Missing docstring in public method
405: D102 Missing docstring in public method
409: D102 Missing docstring in public method
413: D102 Missing docstring in public method
417: D102 Missing docstring in public method
422: D101 Missing docstring in public class
424: D102 Missing docstring in public method
446:32: W602 deprecated form of raising exception
450:30: W601 .has_key() is deprecated, use 'in'
475:32: W602 deprecated form of raising exception
481:52: E261 at least two spaces before inline comment
482:13: E266 too many leading '#' for block comment
502:36: E231 missing whitespace after ','
507:52: E201 whitespace after '['
547:28: W601 .has_key() is deprecated, use 'in'
554:28: W601 .has_key() is deprecated, use 'in'
575:28: W601 .has_key() is deprecated, use 'in'
589: D400 First line should end with a period (not 'r')
595: D102 Missing docstring in public method
597:17: W601 .has_key() is deprecated, use 'in'
605: D102 Missing docstring in public method
629:54: E701 multiple statements on one line (colon)
640:54: E701 multiple statements on one line (colon)
651:54: E701 multiple statements on one line (colon)
655: D102 Missing docstring in public method
666: D102 Missing docstring in public method
672: D402 First line should not be the function's "signature"
676: D102 Missing docstring in public method
679: D102 Missing docstring in public method
682: D402 First line should not be the function's "signature"
692: D210 No whitespaces allowed surrounding docstring text
692: D400 First line should end with a period (not 'd')
699: D210 No whitespaces allowed surrounding docstring text
699: D400 First line should end with a period (not 's')
706: D402 First line should not be the function's "signature"
706: D209 Multi-line docstring closing quotes should be on a separate line
706: D400 First line should end with a period (not 'e')
706: D205 1 blank line required between summary line and description (found 0)
711: D102 Missing docstring in public method
717: D102 Missing docstring in public method
721: D202 No blank lines allowed after function docstring (found 1)
742:1: E302 expected 2 blank lines, found 1
742: D202 No blank lines allowed after function docstring (found 1)
742: D400 First line should end with a period (not 'e')
742: D205 1 blank line required between summary line and description (found 0)
763:35: E231 missing whitespace after ','
791:5: E129 visually indented line with same indent as next logical line
806:1: E302 expected 2 blank lines, found 0
806: D205 1 blank line required between summary line and description (found 0)
814:5: E266 too many leading '#' for block comment
823:5: E266 too many leading '#' for block comment
827:5: E266 too many leading '#' for block comment
832:5: E266 too many leading '#' for block comment
837:5: E266 too many leading '#' for block comment
840:5: E266 too many leading '#' for block comment
844:13: E266 too many leading '#' for block comment
845:13: E266 too many leading '#' for block comment
851:13: E266 too many leading '#' for block comment
853:9: E266 too many leading '#' for block comment
857:13: E266 too many leading '#' for block comment
860:9: E266 too many leading '#' for block comment
865:5: E266 too many leading '#' for block comment